Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#42469 closed defect (fixed)

mozjs17 @17.0.0_2: fails with ccache

Reported by: cooljeanius (Eric Gallager) Owned by: juanrgar (Juan R. García Blanco)
Priority: Normal Milestone:
Component: ports Version: 2.2.1
Keywords: Cc:
Port: mozjs17

Description

I have ccache turned on in my macports.conf file. Relevant part of config.log:

configure:2679: checking whether the C compiler (/opt/local/bin/ccache -pipe -Os -arch x86_64 -L/opt/local/lib -Wl,-headerpad_max_install_names -arch x86_64) works
configure:2695: /opt/local/bin/ccache -o conftest -pipe -Os -arch x86_64 -I/opt/local/include -L/opt/local/lib -Wl,-headerpad_max_install_names -arch x86_64 conftest.c  1>&5
ccache: invalid option -- o

Attachments (3)

config.log (1.7 KB) - added by cooljeanius (Eric Gallager) 10 years ago.
config.log for mozjs17 (x86_64)
main.log (12.8 KB) - added by cooljeanius (Eric Gallager) 10 years ago.
full main.log for mozjs17 failure
ccache-Portfile.diff (353 bytes) - added by juanrgar (Juan R. García Blanco) 10 years ago.
Disable ccache

Download all attachments as: .zip

Change History (11)

Changed 10 years ago by cooljeanius (Eric Gallager)

Attachment: config.log added

config.log for mozjs17 (x86_64)

Changed 10 years ago by cooljeanius (Eric Gallager)

Attachment: main.log added

full main.log for mozjs17 failure

comment:1 Changed 10 years ago by cooljeanius (Eric Gallager)

Just to confirm that this is indeed an issue with using ccache, I turned off ccache and then tried to build it again, and this time the build completed successfully.

comment:2 Changed 10 years ago by cooljeanius (Eric Gallager)

Seeing as r117087 fixed the email address for the maintainer of mozjs17, could someone fix the email address that I cc-ed here, too?

comment:3 Changed 10 years ago by larryv (Lawrence Velázquez)

Cc: juanrgar@… removed
Owner: changed from macports-tickets@… to juanrgar@…

comment:4 Changed 10 years ago by juanrgar (Juan R. García Blanco)

I'm working on this. This should be easy to fix because mozjs17 has an option to build with ccache.

However, what I don't get is why ccache doesn't provide a -o option. I need to read more about ccache in masquerade mode, and how the output file name can be specified.

comment:5 Changed 10 years ago by cooljeanius (Eric Gallager)

The easy thing to do would be to just set configure.ccache no in the Portfile...

Changed 10 years ago by juanrgar (Juan R. García Blanco)

Attachment: ccache-Portfile.diff added

Disable ccache

comment:6 Changed 10 years ago by juanrgar (Juan R. García Blanco)

The problem is macports uses ccache as standalone, and not as masquerade. However, mozjs17 has specific options for building using ccache, so I'll try to investigate a little more if we can get this working, e.g. if we could know that mozjs17 is being build with ccache we could adjust configure options...

comment:7 Changed 10 years ago by dbevans (David B. Evans)

Resolution: fixed
Status: newclosed

ccache patch committed in r117250.

comment:8 in reply to:  7 Changed 10 years ago by cooljeanius (Eric Gallager)

Replying to devans@…:

ccache patch committed in r117250.

Thanks!

Note: See TracTickets for help on using tickets.