-
Notifications
You must be signed in to change notification settings - Fork 41
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
Test #398
Test #398
Conversation
I'm not opposed to these changes, but I don't know for what reason you've submitted them. |
This was an error from me while testing GitHub. I apologize for the inconvenience. |
I apologize for the inconvenience.
Don't apologize, I'm delighted someone is doing this. Yes, I pretty much
gave up on the 10.5 SDK, so that had rotted and I got lazy. All I ask is
to try to do pull requests in chunks so I don't have a huge clot to review
all at once. Feel free to open issues for those with the PRs attached and
I'll go through them.
I suspect the major issue you're seeing with 38ESR and 45ESR is that the
JavaScript backend changed dramatically and I did a number of hacks to the
engine to get it to work with PowerPC. Those will have to be #ifdef __ppc__
or something. If you look at the 45.9 changesets, you should be able to
see exactly where I did this.
Obviously it goes without saying that whatever changes you make can't
break building against the 10.4 SDK or on PowerPC, but other than that,
I'm very happy to hear someone has taken this on. Please ask if you need
advice or suggestions on anything.
|
Hello,
This probably sounds great (and I hope that it will be) but I cannot offer guaranties that the Intel version will be released soon or will simply be. Please understand that TenFourFox is not my principal activity but this publication is my way to thank you from saving my 10.5 MacBook. Also, please do not make official annoncement (as you already did on your blogspot) because I do not want to give fake hope to people following you. A last note: the name visible on my GitHub account is not my true name, I'm just an anonymous enthusiast and contributor. Again, thank you for all your hard work in backward-compatibility and good continuation with TenFourFox. |
No worries, I'll not make any more public references until/if there is a release. That sounds like a really complete set of builds, and very promising! Is the difference the SDK each version is compiled against? Some of the more recent updates the last day or two have PowerPC-specific stuff I added to them. I've tried to mark this, but there might be some unexpected I don't mind working in the way you suggest, though there may be some additional merge overhead on your end. Whatever you think best. Obviously, if this comes to fruition, like the old Intel 17.0.2 build it would be offered with absolutely no support so as not to burden you with that. |
My new branch is available here: http://github.com/OlgaTPark/tenfourfox/tree/Intel and I already uploaded my first commit on it. Don't hesitate if you have questions about my work. For TenFour|Five|SixFox builds, I intend to use conditional compilation with As I said before, I'm checking nearly every change you made to the original Firefox code and if I need to disable some PowerPC-specific stuff on Intel, I'll do it. Unfortunately, I do not want to update my branch to TenFourFox FPR before I'll reach Source Parity with your TenFourFox 45.9 in order to have something perfect (and easy to merge)… The remainder will come laaaater… Thank you for allowing me to make this possible. |
Thank you for allowing me to make this possible.
No, thank *you* for doing it!
|
No description provided.