Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Current status? #13

Closed
mirh opened this issue Mar 25, 2018 · 3 comments
Closed

Current status? #13

mirh opened this issue Mar 25, 2018 · 3 comments

Comments

@mirh
Copy link

mirh commented Mar 25, 2018

Again @gocha @DocSkellington @pocokhc (hope I'm not to bother, otherwise just unsubscribe)
Follows PCSX2#1923


Hi.
I was wondering how much you still have to be considered complete?
AFAIK this is the most recent tas/lua fork of pcsx2, and I (we?) were wondering if you were planning to merge upstream.

@xTVaser
Copy link
Owner

xTVaser commented Mar 25, 2018

I think a merge with upstream would be ideal. I don't know of any critically breaking TAS bugs at the moment, and other than #12 Lua seems to be working as well as it has been since it was added (I havnt done much work on the Lua side of things myself, just attempting to fix bugs).

The main concern that comes to my mind before a merge is:

  • Documentation / Wiki articles are quite different and not sure where the main PCSX2 guys would want it to live
    • The root readme has been heavily modified in this fork so we would need to break that off into a separate readme as well as add our wiki articles and clean them up to pcsx2's

Other than that, the TAS tools are working and its just quality of life improvements and small bug fixes going forward so I see no reason to hold off on the merge after we figure out the best way to go in terms of the docs.

I've also kinda heavily modified the main solution file to make building and debugging easier for us, so I would want to exclude that from the merge.

@mirh
Copy link
Author

mirh commented Mar 25, 2018

When you want, the honor of a PR is all yours.

Documentation / Wiki articles are quite different and not sure where the main PCSX2 guys would want it to live

Github wikis are like totally unconnected repos AFAIU.
(guess like "Building from Source" is quite appreciated also)

I've also kinda heavily modified the main solution file to make building and debugging easier for us, so I would want to exclude that from the merge.

Or you could drop some idea on PCSX2#2342

@xTVaser
Copy link
Owner

xTVaser commented Mar 25, 2018

Yes wikis are separate repos you're right. I don't really have any suggestions for the solution files, it was more just me deleting things that were not relevant to debugging these tools.

I'll work on cleaning things up in preparation for that PR soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants