[MacPorts] #24936: makeicns should ensure it's UsingTheRightCompiler
#24936: makeicns should ensure it's UsingTheRightCompiler -------------------------------------+-------------------------------------- Reporter: ryandesign@… | Owner: mk-macports@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 1.8.2 Keywords: | Port: makeicns -------------------------------------+-------------------------------------- makeicns should ensure it's UsingTheRightCompiler: {{{ ---> Fetching makeicns ---> Verifying checksum(s) for makeicns ---> Extracting makeicns ---> Configuring makeicns ---> Building makeicns Error: Target org.macports.build returned: shell command " cd "/opt/local/var/macports/build/_Users_rschmidt_macports_dports_graphics_makeicns/work/makeicns" && /usr/bin/nice -n 10 /usr/bin/make -j2 " returned error 2 Command output: cc -fpascal-strings -O2 -arch ppc -arch i386 -c -o makeicns.o makeicns.m cc -fpascal-strings -O2 -arch ppc -arch i386 -c -o IconFamily.o IconFamily.m cc -fpascal-strings -O2 -arch ppc -arch i386 -c -o IconFamily.o IconFamily.m cc: Error: You should be using ${configure.cc} See http://trac.macports.org/wiki/UsingTheRightCompiler make: *** [IconFamily.o] Error 1 make: *** Waiting for unfinished jobs.... cc -fpascal-strings -O2 -arch ppc -arch i386 -c -o makeicns.o makeicns.m cc: Error: You should be using ${configure.cc} See http://trac.macports.org/wiki/UsingTheRightCompiler make: *** [makeicns.o] Error 1 }}} -- Ticket URL: <http://trac.macports.org/ticket/24936> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24936: makeicns should ensure it's UsingTheRightCompiler -------------------------------------+-------------------------------------- Reporter: ryandesign@… | Owner: mk-macports@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 1.8.2 Keywords: | Port: makeicns -------------------------------------+-------------------------------------- Comment(by MK-MacPorts@…): Hi Ryan, is my quick fix accepted? Greets, Marko -- Ticket URL: <http://trac.macports.org/ticket/24936#comment:1> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24936: makeicns should ensure it's UsingTheRightCompiler --------------------------------------+------------------------------------- Reporter: ryandesign@… | Owner: mk-macports@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 1.8.2 Resolution: fixed | Keywords: Port: makeicns | --------------------------------------+------------------------------------- Changes (by ryandesign@…): * status: new => closed * resolution: => fixed Comment: Sorry, I didn't see your attachment until you posted your comment just now. Trac sends email notification for new comments, but not for new attachments. If you actually set up the script described in the [wiki:UsingTheRightCompiler#testing How to test] section of the wiki article, you'll see that even after applying your patch, the error message is still triggered. MacPorts sets the CC, CXX, etc. environment variables only during the configure phase, and since makeicns doesn't have a configure phase, it doesn't see these variables and you have to help it out by manually setting the variables during the build phase, as described in [wiki:UsingTheRightCompiler#nonstandard-ports the section on ports with nonexistent configure scripts]. Fixed in r67927. -- Ticket URL: <http://trac.macports.org/ticket/24936#comment:2> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24936: makeicns should ensure it's UsingTheRightCompiler --------------------------------------+------------------------------------- Reporter: ryandesign@… | Owner: mk-macports@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 1.8.2 Resolution: fixed | Keywords: Port: makeicns | --------------------------------------+------------------------------------- Comment(by MK-MacPorts@…): Ooops, behavior of the configure/build steps seems to be different on different systems... (I did test whether install threw any warning, which it didn't before I submitted my patch.) Thanks for your help!!! -- Ticket URL: <http://trac.macports.org/ticket/24936#comment:3> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24936: makeicns should ensure it's UsingTheRightCompiler --------------------------------------+------------------------------------- Reporter: ryandesign@… | Owner: mk-macports@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 1.8.2 Resolution: fixed | Keywords: Port: makeicns | --------------------------------------+------------------------------------- Comment(by ryandesign@…): The behavior should be the same on any system, provided you've downloaded the script mentioned at the end of UsingTheRightCompiler and modified the binpath in your macports.conf to point to where it is... -- Ticket URL: <http://trac.macports.org/ticket/24936#comment:4> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24936: makeicns should ensure it's UsingTheRightCompiler --------------------------------------+------------------------------------- Reporter: ryandesign@… | Owner: mk-macports@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 1.8.2 Resolution: fixed | Keywords: Port: makeicns | --------------------------------------+------------------------------------- Comment(by MK-MacPorts@…): Thanks again, Ryan, no, of course I hadn't tested said thing... Now I did and I got the desired behavior! Building ceased to function with this change. '''I guess this should be part of a standard MacPorts installation, don't you think?''' Greets, Marko -- Ticket URL: <http://trac.macports.org/ticket/24936#comment:5> MacPorts <http://www.macports.org/> Ports system for Mac OS
#24936: makeicns should ensure it's UsingTheRightCompiler --------------------------------------+------------------------------------- Reporter: ryandesign@… | Owner: mk-macports@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 1.8.2 Resolution: fixed | Keywords: Port: makeicns | --------------------------------------+------------------------------------- Comment(by ryandesign@…): Replying to [comment:5 MK-MacPorts@…]:
'''I guess this should be part of a standard MacPorts installation, don't you think?''' Not particularly at this point, since there are sooo many ports that currently don't ensure they're UsingTheRightCompiler, and quite a few ports where I have no idea how to fix it even because their build scripts are so convoluted. I think it's ok to leave this as an option for adventurous people to enable when they feel like finding and fixing stuff that's broken. I myself alternate between having it enabled (when I have time to file lots of tickets) and disabled (when I just want to get the darn ports installed).
-- Ticket URL: <http://trac.macports.org/ticket/24936#comment:6> MacPorts <http://www.macports.org/> Ports system for Mac OS
participants (1)
-
MacPorts