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)

main.log (457.1 KB) - added by rupapara.prashant@… 12 years ago.

Download all attachments as: .zip

Change History (9)

Changed 12 years ago by rupapara.prashant@…

Attachment: main.log added

comment:1 Changed 12 years ago by rupapara.prashant@…

sudo port install irsim configure.compiler=llvm-gcc-4.2 works.... so perhaps you need to change compiler option in portfile.

comment:2 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: ryandesign@… added
Owner: changed from macports-tickets@… to and.damore@…
Summary: irsimirsim: 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:3 Changed 12 years ago by rupapara.prashant@…

Hey, Have you applied changes to portfile?

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: newassigned

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.

Last edited 12 years ago by anddam (Andrea D'Amore) (previous) (diff)

comment:8 Changed 12 years ago by anddam (Andrea D'Amore)

Resolution: fixed
Status: assignedclosed

Committed r97986, disabled macosx definition on 10.8 and removed /sw.

Note: See TracTickets for help on using tickets.