Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#18921 closed defect (fixed)

subversion @1.6.0 +tools broken

Reported by: raimue (Rainer Müller) Owned by: danielluke (Daniel J. Luke)
Priority: Normal Milestone:
Component: ports Version: 1.8.0
Keywords: Cc: ryandesign (Ryan Schmidt), 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 (1)

patch-subversion+tools.diff (783 bytes) - added by raimue (Rainer Müller) 12 years ago.
revised patch

Download all attachments as: .zip

Change History (10)

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

Cc: ryandesign@… added

Cc Me!

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

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 12 years ago by raimue (Rainer Müller)

Attachment: patch-subversion+tools.diff added

revised patch

comment:3 Changed 12 years ago by raimue (Rainer Müller)

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

comment:4 Changed 12 years ago by vinc17@…

Cc: vinc17@… added

comment:5 Changed 12 years ago by steven@…

Cc: steven@… added

Cc Me!

comment:6 Changed 12 years ago by daniel@…

Cc: daniel@… added

Cc Me!

comment:7 Changed 12 years ago by daniel@…

Cc: daniel@… removed

Cc Me!

comment:8 Changed 12 years ago by blair (Blair Zajac)

Resolution: fixed
Status: newclosed

Fixed in r48489.

comment:9 Changed 12 years ago by (none)

Milestone: Port Bugs

Milestone Port Bugs deleted

Note: See TracTickets for help on using tickets.