Opened 12 years ago
Closed 12 years ago
#36250 closed defect (fixed)
irsim: error: expected a platform name, e.g., 'macosx'
Reported by: | rupapara.prashant@… | Owned by: | anddam (Andrea D'Amore) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.1.2 |
Keywords: | mountainlion | Cc: | ryandesign (Ryan Carsten Schmidt) |
Port: | irsim |
Description
Attachments (1)
Change History (9)
Changed 12 years ago by rupapara.prashant@…
comment:1 Changed 12 years ago by rupapara.prashant@…
comment:2 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)
Cc: | ryandesign@… added |
---|---|
Owner: | changed from macports-tickets@… to and.damore@… |
Summary: | irsim → irsim: error: expected a platform name, e.g., 'macosx' |
The error I see in the log you attached is:
error: expected a platform name, e.g., 'macosx'
I just fixed this problem in magic recently so hopefully the fix I applied there can be used for irsim as well. The problem is that a #define "macosx" is being set by the project, and this conflicts with directives in the system headers on at least Mountain Lion. The fix is to remove the custom macosx
#define and to use the __APPLE__
#define Apple already provides.
I can confirm that using llvm-gcc-4.2 works around the problem but I'm surprised that works. I think we should not employ that workaround in the port; instead we should patch the software just as I patched magic already, and submit that patch to the developers.
First we should update irsim to the latest version. I just fixed the livecheck.
We should also apply additional patches to irsim that are already in the magic port, including removing "-I/sw/include" from the build system.
comment:4 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)
No, I have not yet had a chance to look into it.
comment:5 Changed 12 years ago by anddam (Andrea D'Amore)
Status: | new → assigned |
---|
comment:6 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)
Andrea, the main thing we need to do is take the fix from r97859 and port it to irsim. Do you want to look into it or should I?
comment:7 Changed 12 years ago by anddam (Andrea D'Amore)
Keywords: | mountainlion added |
---|
I will, the macro is only checked in base/mem.c .
The issue seems mountainlion-specific since I cannot reproduce on 10.7 with xcode 4.4.1, furthermore from the reporter's main log it seems configure script is not finding X libraries.
comment:8 Changed 12 years ago by anddam (Andrea D'Amore)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Committed r97986, disabled macosx definition on 10.8 and removed /sw.
sudo port install irsim configure.compiler=llvm-gcc-4.2 works.... so perhaps you need to change compiler option in portfile.