-
Notifications
You must be signed in to change notification settings - Fork 3
Merge latest code from @kanjitalk755 #14
Comments
I'm talking with one of the other devs about this. I've just discovered that there are a LOT of symlinked files between the 2 projects, and so at the moment the only sensible thing to do appears to be splitting off those common elements into a standalone This is more or less the kind of rabbit hole I expected to encounter, but I'm prepared to sort it out. In the immediate term, my plan is to start over from @kanjitalk755's repo (fix the symlinks, split the resulting repo), and then merge in the work that we've done here. I'm going to peek at that now |
Thank you very much indeed for the information! |
I've captured stuff in #15 and my plan for now is to avoid accepting requests on any of those files unless/untill we establish a way to build those all from a shared location. That's going to put a lot of focus on refactoring instead of moving forward with new features, but I don't think the current setup is healthy for long-term maintenance. |
I fully agree with you! I did add to #15 some historical context which could be useful. |
This is done, but it's very likely that I broke the builds in the process and will have to fix those as a top priority. |
@ianfixes Can you also include the latest commit that provides a different icon for SheepVM bundles? |
Hmm, looks like that got submitted after I started work on combining repos and splitting projects. I'm just going to pull the files in directly for now, and then my main concern will be the build tasks listed in kanjitalk755/macemu#14 |
Done: 07cac57 |
Refer to discussion at:
kanjitalk755/macemu#14
Bring in these edits, as they are the most current development of this project.
The text was updated successfully, but these errors were encountered: