Opened 7 weeks ago

Closed 6 weeks ago

#70935 closed defect (fixed)

expect broken by the last revbump

Reported by: barracuda156 Owned by: saagarjha (Saagar Jha)
Priority: Normal Milestone:
Component: ports Version: 2.10.1
Keywords: haspatch Cc: saagarjha (Saagar Jha), herbygillot (Herby Gillot), ballapete (Peter "Pete" Dyballa), huyz (Huy Z), cooljeanius (Eric Gallager)
Port: expect

Description

This is how it installs now after removing --disable-shared: 

36-100% otool -L /opt/local/bin/expect
/opt/local/bin/expect:
	libexpect5.45.4.dylib (compatibility version 5.45.4, current version 5.45.4)
	/opt/local/lib/libtcl8.6.dylib (compatibility version 8.6.0, current version 8.6.14)
	/usr/lib/libgcc_s.1.dylib (compatibility version 1.0.0, current version 1.0.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 117.0.0)

Expectedly, the port is broken in result: 

--->  Scanning binaries for linking errors
Could not open libexpect5.45.4.dylib: Error opening or reading file (referenced from /opt/local/bin/expect)
libexpect5.45.4.dylib seems to be referenced using a relative path. This may be a problem with its canonical library name and require the use of install_name_tool(1) to fix.
--->  Found 1 broken file, matching files to ports
--->  Found 1 broken port, determining rebuild order

Change History (7)

comment:1 Changed 7 weeks ago by barracuda156

Dylib installs with the wrong path too: 

36-100% otool -L /opt/local/lib/expect5.45.4/libexpect5.45.4.dylib
/opt/local/lib/expect5.45.4/libexpect5.45.4.dylib:
	libexpect5.45.4.dylib (compatibility version 5.45.4, current version 5.45.4)
	/usr/lib/libgcc_s.1.dylib (compatibility version 1.0.0, current version 1.0.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 117.0.0)

comment:2 Changed 7 weeks ago by ballapete (Peter "Pete" Dyballa)

Cc: ballapete added

comment:3 Changed 7 weeks ago by ryandesign (Ryan Carsten Schmidt)

Keywords: haspatch added

I think this is supposed to be fixed by ​https://github.com/macports/macports-ports/pull/25920

comment:4 Changed 7 weeks ago by huyz (Huy Z)

Cc: huyz added

comment:5 in reply to:  3 Changed 7 weeks ago by barracuda156

Replying to ryandesign:

I think this is supposed to be fixed by ​https://github.com/macports/macports-ports/pull/25920

It does, but unmerged still.

comment:6 Changed 6 weeks ago by cooljeanius (Eric Gallager)

Cc: cooljeanius added

comment:7 Changed 6 weeks ago by saagarjha (Saagar Jha)

Owner: set to saagarjha
Resolution: → fixed
Status: new → closed

In 8510e773ae4122ca2c6cb83dbdd2d5c99c1cebb8/macports-ports (master):

expect: Fix install name for libexpect dylib

Closes: #70935

Note: See TracTickets for help on using tickets.