-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
colm & ragel: updates of both ports and fixes for PPC #15762
Conversation
So
|
the problem with the test is that it is not being built for the build arch. probably missing arch flags I would guess. |
what is wrong with building the docs exactly? |
are we doomed to have @barracuda156 add this to 10,000 ports to support rosetta? This is added by the muniversal PG I believe, but that is not being used here for his pure cross build. Also, I believe this will not work if muniversal is added (but I haven’t tested that). platform darwin 10 powerpc { |
@kencu TBH I would rather find a better and universal fix, than adding this manually (whether in Macports or locally). If you could suggest how to fix that in Macports base, that will be very much preferred (by myself too). |
I can't change base for this, but if there is suitable motivation, it might be done. add that for all builds when the build is not being done by cmake, meson, or other non-autotools builds maybe. Unlikely to happen, but might as well ask as adding the same exact block to every autotools build is kinda silly. |
This also failed on build bots, not just for me. |
@kencu This is by the way genuinely Rosetta-specific problem (on 10.6 PPC the triple is set correctly). But I think it makes sense to have Rosetta fixed, since that is a more accessible environment for anyone wishing to build/test smth for PPC (at least @mascguy Any thoughts? @kencu is right, we rather have a neat general fix for ports that do not use |
so ideally the build line calling asciidoc needs fixing, either by us or upstream. Presumably that is in:
|
It's not really even wrong. The machine you're building on really is x86/x64. If you want to build code for a different system than you're running on, the correct option to set is technically |
@jmroot The physical cpu is x86, but from what I understand this is not a scenario of cross-compiling, since Rosetta emulated PPC cpu. That is, we do not use Intel compilers to build for PPC (cross-compilation scenario), all software is natively PPC, even Macports itself. |
it is mostly cross compiling, as you are usually calling gcc-4.2 built as intel to build the powerpc code -- which needs to be forced to build cross with this tedious block. when you call gcc7 or gcc12 it is using powerpc code, and an emulator, so not cross compiling. so doesn't need this block. likely. it's a true, hacky, mess. |
@kencu Turns out that the error @essandess No solution ever since, right? Strangely, on 10.6.8 it builds fine though. |
Correct. I've been waiting to see if this ever affects |
Update from 10.5.8.
Its universal variant fails on Leopard however:
@kencu @mascguy Should I add |
@essandess
Any idea why on older OS there is no error? UPD. Let me try using latest-commit |
Updated to latest-commit versions from Github, rebased upon master. Let’s see if this works. UPD. Well, no, same story :( |
@mascguy Ready for review, checks passed. |
Description
colm
updated to 0.14.7.ragel
updated to 7.0.4.Type(s)
Tested on
macOS 10.6.8 Server
Xcode 3.2.6
Verification
Have you
port lint --nitpick
?sudo port test
?sudo port -vst install
?