New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #18921 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

subversion @1.6.0 +tools broken

Reported by: raimue@… Owned by: dluke@…
Priority: Normal Milestone:
Component: ports Version: 1.8.0
Keywords: Cc: ryandesign@…, vinc17@…, steven@…
Port: subversion

Description

$ sudo port install subversion +tools
...
--->  Staging subversion into destroot
Error: Target org.macports.destroot returned: error copying "/opt/local/var/macports/build/_Users_raim_src_macports_trunk_dports_devel_subversion/work/subversion-1.6.0/contrib/client-side/svnmucc/svnmucc": no such file or directory
Error: Unable to upgrade port: 1

Patch attached. Not bumping revision since this prevented upgrade/installation for anyone with +tools.

Attachments

patch-subversion+tools.diff (783 bytes) - added by raimue@… 4 years ago.
revised patch

Change History

comment:1 Changed 4 years ago by ryandesign@…

  • Cc ryandesign@… added

Cc Me!

comment:2 Changed 4 years ago by ryandesign@…

The point of doing the commands in this order

eval delete [glob ${worksrcpath}/contrib/client-side/{svnmucc{,.{l,}o},.libs}] 
copy ${worksrcpath}/contrib ${destroot}${prefix}/share/${name}/contrib

was so that svnmucc's temporary build artifacts don't get installed when the copy command is run. But in your patch you moved the eval delete after the copy. So either the build artifacts are now being unnecessarily installed, or if they aren't, then the eval delete isn't necessary anymore.

Changed 4 years ago by raimue@…

revised patch

comment:3 Changed 4 years ago by raimue@…

Agreed, I missed that. I have updated the patch which does the copy after modifications.

comment:4 Changed 4 years ago by vinc17@…

  • Cc vinc17@… added

comment:5 Changed 4 years ago by steven@…

  • Cc steven@… added

Cc Me!

comment:6 Changed 4 years ago by daniel@…

  • Cc daniel@… added

Cc Me!

comment:7 Changed 4 years ago by daniel@…

  • Cc daniel@… removed

Cc Me!

comment:8 Changed 4 years ago by blair@…

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

Fixed in r48489.

comment:9 Changed 4 years ago by anonymous

  • Milestone Port Bugs deleted

Milestone Port Bugs deleted

Note: See TracTickets for help on using tickets.