Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#53490 closed defect (fixed)

port diagnose fails on symlink

Reported by: fracai Owned by: jmroot (Joshua Root)
Priority: Normal Milestone: MacPorts 2.4.1
Component: base Version: 2.4.0
Keywords: Cc: pixilla (Bradley Giesbrecht), mkae (Marko Käning)
Port:

Description

$ sudo port diagnose
…
Checking for file '/opt/local/lib/pkgconfig/texluajit.pc' on disk... [SUCCESS]
Checking for file '/opt/local/libexec/texlive/binaries/a2ping' on disk... [FAILED]
Checking for file '/opt/local/libexec/texlive/binaries/a5toa4' on disk... [FAILED]
Checking for file '/opt/local/libexec/texlive/binaries/adhocfilelist' on disk... [FAILED]
Checking for file '/opt/local/libexec/texlive/binaries/afm2afm' on disk... [FAILED]
Checking for file '/opt/local/libexec/texlive/binaries/afm2pl' on disk... [SUCCESS]
…

All of the failed lines represent symlinks to other files that do exist on disk.

I also note that these failures aren't actually reported unless I redirect the output and without that redirect, there's no indication that anything has failed. I'm not sure if that indicates the file is actually detected (unlikely) or if that's a different issue where the failures aren't reported.

Change History (7)

comment:1 Changed 6 years ago by mf2k (Frank Schima)

Keywords: diagnose symlink removed

comment:2 Changed 6 years ago by pixilla (Bradley Giesbrecht)

Cc: pixilla added

comment:3 Changed 6 years ago by mkae (Marko Käning)

Cc: mkae added

comment:4 Changed 6 years ago by mkae (Marko Käning)

Owner: set to admin@…
Status: newassigned

(I am not sure whether assigning this to admin@mpo is actually necessary, but lately all new tickets have an empty owned by field...)

comment:5 Changed 6 years ago by jmroot (Joshua Root)

Owner: admin@… deleted

No, assigning to admin@ is only correct for tickets that need the attention of a sysadmin (so usually server/hosting component).

comment:6 Changed 6 years ago by jmroot (Joshua Root)

Owner: set to jmroot
Resolution: fixed
Status: assignedclosed

In 103d6f6/macports-base:

Port diagnose file check fix

Don't report failure before checking whether a port's apparently
missing file is actually a link.

Fixes: #53490

comment:7 Changed 6 years ago by jmroot (Joshua Root)

Milestone: MacPorts 2.4.1
Note: See TracTickets for help on using tickets.