Opened 12 years ago

Closed 12 years ago

#19879 closed defect (fixed)

ardour2 doesn't use ${configure.cc}

Reported by: ryandesign (Ryan Schmidt) Owned by: dbevans (David B. Evans)
Priority: Normal Milestone:
Component: ports Version: 1.7.1
Keywords: Cc:
Port: ardour2

Description

ardour2 uses "gcc" instead of the value in ${configure.cc}.

See UsingTheRightCompiler.

Attachments (1)

ardour2-gcc-4.3.diff (809 bytes) - added by ryandesign (Ryan Schmidt) 12 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 12 years ago by dbevans (David B. Evans)

Resolution: fixed
Status: newclosed

Updated to version 2.8 which needs gcc43 to build correctly. Scons requires needs the compiler specified in the env vars for each run. See Portfile for details. Let me know if you see a better approach but this works and invokes the desired compiler(s) correctly.

comment:2 Changed 12 years ago by dbevans (David B. Evans)

Update committed in r51938 by the way.

Changed 12 years ago by ryandesign (Ryan Schmidt)

Attachment: ardour2-gcc-4.3.diff added

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

Resolution: fixed
Status: closedreopened

Thanks! After a lot of building I still have:

os.chdir('gtk2_ardour')
cpp -E -P ardour.menus.in ardour.menus
cpp -E -P ardour.menus.in ardour.menus
cpp: Error: You should be using /usr/bin/cpp-*:
See http://trac.macports.org/wiki/UsingTheRightCompiler
os.chdir('/mp/var/macports/build/_Users_rschmidt_macports_dports_audio_ardour2/work/ardour-2.8')
scons: *** [gtk2_ardour/ardour.menus] Error 1
scons: building terminated because of errors.

So CPP still needs to be set somehow.

As for requiring gcc 4.3, I attached a small patch for you.

comment:4 Changed 12 years ago by dbevans (David B. Evans)

I don't see any of this at all on tiger ppc using MacPorts 1.7.1. What versions/platform are you using and where is this error message about UsingTheRightCompiler coming from?

Thanks for the patch by the way.

comment:5 Changed 12 years ago by dbevans (David B. Evans)

Never mind. I read to the bottom and see what you are doing. Will install your test, add the patch, include definitions of CPP and see if that fixes it. As you say it takes some compiling time so may be a while.

Sorry for the delay in responding, I didn't see that you had reopened the ticket.

comment:6 Changed 12 years ago by dbevans (David B. Evans)

Resolution: fixed
Status: reopenedclosed

The problem with cpp was that gtk2_ardour/SConscript was invoking it explicitly in a static string without any reference to environment variables.

Committed your patch with slight modifications to include CPP and an additional patch/reinplace to fix the cpp problem in r52654.

Thanks

Note: See TracTickets for help on using tickets.