Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#48431 closed defect (fixed)

mozjs17 @17.0.0_2: configure script causes software to build incorrectly with perl5.22

Reported by: bobmath Owned by: juanrgar@…
Priority: Normal Milestone:
Component: ports Version: 2.3.3
Keywords: Cc: Ionic (Mihai Moldovan), dbevans (David B. Evans)
Port: mozjs17

Description

Installing policykit @0.113_1 from binary or source is failing on OSX 10.10.4. Binary complains about broken files. Config log from source build says that mozjs cannot be found, but mozjs17 port is already installed.

Attachments (6)

script.txt (2.5 KB) - added by bobmath 9 years ago.
output from port install policykit
config.log (35.4 KB) - added by bobmath 9 years ago.
main.log (18.9 KB) - added by bobmath 9 years ago.
port_contents_mozjs17.txt (2.6 KB) - added by bobmath 9 years ago.
port_-v_install_policykit.txt (16.9 KB) - added by bobmath 9 years ago.
milestone.pl.patch (342 bytes) - added by bobmath 9 years ago.
fix for mozjs config script

Download all attachments as: .zip

Change History (25)

Changed 9 years ago by bobmath

Attachment: script.txt added

output from port install policykit

Changed 9 years ago by bobmath

Attachment: config.log added

Changed 9 years ago by bobmath

Attachment: main.log added

comment:1 Changed 9 years ago by Ionic (Mihai Moldovan)

Cc: ionic@… added
Owner: changed from macports-tickets@… to devans@…

Please remember to CC the maintainer, if any, next time.

Dave, maybe a revbump is in order?

N.B.: ${prefix}/lib/pkgconfig/mozjs-17.0.pc exists on my system. Reporter: what does port contents mozjs17 list?

comment:2 Changed 9 years ago by Ionic (Mihai Moldovan)

Also, can you please run sudo port -v install policykit so that we know what binary is broken exactly?

comment:3 Changed 9 years ago by bobmath

/opt/local/lib/pkgconfig/mozjs-..pc

That seems weird. Maybe this is a mozjs issue.

Changed 9 years ago by bobmath

Attachment: port_contents_mozjs17.txt added

Changed 9 years ago by bobmath

comment:4 Changed 9 years ago by Ionic (Mihai Moldovan)

There's a lot of stuff wrong with that list. "1.7" has been stripped all over the place.

port installed mozjs17? I'm on 17.0.0_2 with +universal.

comment:5 Changed 9 years ago by Ionic (Mihai Moldovan)

Yep, the brokenness makes sense, relating to mozjs17 again and its "missing" library... which is "available" on your system, but horribly misnamed.

comment:6 Changed 9 years ago by bobmath

port installed mozjs17

mozjs17 @17.0.0_2 (active)

comment:7 Changed 9 years ago by Ionic (Mihai Moldovan)

Okay, up to date.

Let's reinstall/rebuild this (not sure what your system will do and if archives are available.)

sudo port -nv upgrade --force mozjs17

N.B.: never do something like I just told you if you don't know the consequences.

comment:8 Changed 9 years ago by bobmath

It built from source, and it's done the same thing. There's a lot of

a ./opt/local/lib/libmozjs-..a
a ./opt/local/lib/libmozjs-..dylib

You didn't want the whole log, did you? It's like 3 megs.

comment:9 Changed 9 years ago by bobmath

Mozjs configure script fails with perl5.22. Appears to be a trivial fix. Switched to perl5.20 and everything installed cleanly.

comment:10 Changed 9 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: devans@… added
Owner: changed from devans@… to juanrgar@…
Port: mozjs17 added; policykit removed
Summary: policykit @0.113_1 fails to install, missing mozjs?mozjs17 @17.0.0_2: configure script causes software to build incorrectly with perl5.22

comment:11 Changed 9 years ago by Ionic (Mihai Moldovan)

Ouch, that's not good. We're in the process of moving to perl5.22. Also, it has no formal dependency on perl. How did you switch to perl5.20?

Changed 9 years ago by bobmath

Attachment: milestone.pl.patch added

fix for mozjs config script

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

Many configure scripts use perl, assuming it is available on all systems.

In this case, it probably uses whichever perl the user has installed with the perl5 port.

comment:13 Changed 9 years ago by bobmath

To change perl versions:

port install perl5 +perl5_20

comment:14 Changed 9 years ago by Ionic (Mihai Moldovan)

Yes, that depends on perl5.20 and changes symlinks of binaries and other stuff to the 5.20 version.

The patch looks good to me.

comment:15 Changed 9 years ago by Ionic (Mihai Moldovan)

Have you tested that mozjs builds with the patch and perl5.22?

comment:16 Changed 9 years ago by bobmath

Since my macports-fu is not very good, I just made the change manually in /opt/local/var/macports/build and ran 'port build mozjs17' and it seemed to work.

comment:17 Changed 9 years ago by Ionic (Mihai Moldovan)

Resolution: fixed
Status: newclosed

Added patch in r138958. Thanks!

comment:18 Changed 9 years ago by dbevans (David B. Evans)

Similar issue with mozjs24 fixed in r139054.

comment:19 Changed 9 years ago by Ionic (Mihai Moldovan)

Thanks! I didn't bother to check the other mozjs ports.

Note: See TracTickets for help on using tickets.