New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #29704 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

go portfile creates bad symlinks

Reported by: cgilmour@… Owned by: singingwolfboy@…
Priority: Normal Milestone:
Component: ports Version: 1.9.2
Keywords: haspatch Cc: ryandesign@…
Port: go

Description

Hi,

Installing "go" creates a symbolic link /src/Make.*, instead of a separate link for each individual make file.

116 # gomake command required makefiles

117 ln -s ${prefix}/src/${name}/${name}-${version}/src/Make.* ${destroot}${prefix}/src

Seems to be related to changes from Ticket #29321

I can't find an example in other Portfiles that tries to make symlinks like this, so I don't have a fix/patch.

(I changed mine to use xinstall instead of making symbolic links, plus other changes to install without root privileges. I think the package maintainer might do it a different way instead)

Thanks Caleb

Attachments

go-ln.diff (896 bytes) - added by ryandesign@… 4 years ago.
proposed patch
go-ln.2.diff (957 bytes) - added by ryandesign@… 4 years ago.
fixed proposed patch

Change History

comment:1 Changed 4 years ago by ryandesign@…

  • Cc ryandesign@… added
  • Owner changed from macports-tickets@… to singingwolfboy@…
  • Port set to go

The fix is to use an "eval glob" construct.

comment:2 Changed 4 years ago by ryandesign@…

Or possibly an fs-traverse loop.

Changed 4 years ago by ryandesign@…

proposed patch

comment:3 Changed 4 years ago by ryandesign@…

  • Keywords haspatch added

Or neither of those. ("eval ... [glob ...]" would work if MacPorts' ln supported creating multiple links at once, like the shell's ln does, but it doesn't seem to.) The attached patch uses a "foreach ... [glob ...]" and seems to work. Perhaps this should be committed together with the fix in #29706.

comment:4 Changed 4 years ago by ryandesign@…

No wait, this patch is wrong, needs to glob the files in the destroot...

Changed 4 years ago by ryandesign@…

fixed proposed patch

comment:5 Changed 4 years ago by ryandesign@…

Here's the fixed version.

comment:6 Changed 4 years ago by singingwolfboy@…

  • Status changed from new to assigned

Thanks! Verifying your patch on my computer now: it if builds cleanly and fixes the issue, I'll commit it.

comment:7 Changed 4 years ago by singingwolfboy@…

  • Status changed from assigned to closed
  • Resolution set to fixed

Committed in r79186.

Note: See TracTickets for help on using tickets.