Opened 21 years ago

Closed 21 years ago

Last modified 21 years ago

#1553 closed defect (fixed)

fetch stage fails if distfile directory is not writable, even if proper distfile is already present

Reported by: opendarwin.org@… Owned by: pguyot (Paul Guyot)
Priority: Normal Milestone:
Component: base Version: 1.0
Keywords: Cc:
Port:

Description

fetch stage of "port" fails if the distfile directory (in this case ${prefix}/var/db/dports/distfiles/ ethereal/) is not writable, even though the proper distfile already exists in the directory. I've moved the "writable directory" check into the "file exists and fetch" loop, attaching a diff. With that change, I get output of:


port -v -d fetch DEBUG: com.apple.package registered provides 'package', a pre-existing procedure. Target override will not be provided DEBUG: Executing com.apple.main (ethereal) ---> Fetching ethereal DEBUG: Executing com.apple.fetch (ethereal) ---> ethereal-0.10.1.tar.bz2 doesn't seem to exist in /opt/local/var/db/dports/distfiles/ethereal Error: Target com.apple.fetch returned: /opt/local/var/db/dports/distfiles/ethereal must be writable Warning: the following items did not execute (for ethereal): com.apple.fetch


This seems to be a bit more useful as well, as it shows the file that its going to try to fetch, and where

it has tried to find it, and then indicates the error.

Tested on 10.3.2, only on net/ethereal that I'm working on.

-eric

Attachments (1)

portfetch.tcl.diff (968 bytes) - added by opendarwin.org@… 21 years ago.
move the "writable directory" check to after checking for distfile existence

Download all attachments as: .zip

Change History (3)

Changed 21 years ago by opendarwin.org@…

Attachment: portfetch.tcl.diff added

move the "writable directory" check to after checking for distfile existence

comment:1 Changed 21 years ago by pguyot (Paul Guyot)

Owner: changed from darwinports-bugs@… to pguyot@…

comment:2 Changed 21 years ago by pguyot (Paul Guyot)

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.