Paths

Table of Contentst

Differential D42156

whereis(1): Respect PORTSDIR variable
Needs ReviewPublic
Actions

Authored by fernape on Oct 11 2023, 11:12 AM.

Details

Reviewers

None

Group Reviewers

Src Committers

Summary

Respect PORTSDIR variable for those who have the ports collection in a different
place than /usr/ports. PORTSDIR is a very common variable used in the ports
framework and in /etc/make.conf among other places.

Add note to manual page.
.Dd to be bumped at commit time.

Note that this same behavior can be achieved by doing something like:

export PORTSDIR=~/FreeBSD-repos/ports
whereis -S $PORTSDIR/** -f converseen

Assuming the shell is able to expand the **, but it is much less convenient.

Test Plan

Assuming the ports tree is in $HOME/FreeBSD-repos/ports and that there is no
/usr/ports, if we execute the following:

$ whereis converseen
converseen:

Apply the patch and run the following:

$ PORTSDIR=$HOME/FreeBSD-repos/ports ./whereis converseen
converseen: /home/fernape/FreeBSD-repos/ports/graphics/converseen

If we create the /usr/ports directory and populate some directories:

$ ls -R /usr/ports/
foo/      graphics/

/usr/ports/foo:
bar/

/usr/ports/foo/bar:

/usr/ports/graphics:
converseen/

/usr/ports/graphics/converseen:
Makefile   distinfo   pkg-descr  pkg-plist

and execute the same command we get the /usr/ports result as expected:

$ PORTSDIR=$HOME/FreeBSD-repos/ports ./whereis converseen
converseen: /usr/ports/graphics/converseen

Asking for all results gives both entries:

$ PORTSDIR=$HOME/FreeBSD-repos/ports ./whereis -a converseen
converseen: /usr/ports/graphics/converseen
/home/fernape/FreeBSD-repos/ports/graphics/converseen

Unsetting the variable works as expected:

$ ./whereis -a converseen
converseen: /usr/ports/graphics/converseen

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 75986
Build 72869: arc lint + arc unit

Event Timeline

fernape created this revision.Oct 11 2023, 11:12 AM

Herald added a subscriber: imp. · View Herald TranscriptOct 11 2023, 11:12 AM

fernape requested review of this revision.Oct 11 2023, 11:12 AM

Harbormaster completed remote builds in B53923: Diff 128545.Oct 11 2023, 11:12 AM

fernape edited the summary of this revision. (Show Details)Oct 11 2023, 11:13 AM

fernape added a reviewer: Src Committers.

patmaddox added a subscriber: patmaddox.Oct 19 2023, 9:44 PM

Ping!

Friendly ping

Another ping!

Herald added a subscriber: ziaee. · View Herald TranscriptAug 17 2025, 3:37 PM

Ping! 🔔

delphij added a subscriber: delphij.Sat, Aug 22, 4:17 PM

delphij added inline comments.

usr.bin/whereis/whereis.c
382–383	Why not simply using `b` here for the assignment? (For readability you might want to set `b` as `NULL` afterward to make it explicit that the ownership is changed, but doing `strdup()` here seems to be redundant, and OOM situation is not handled).

Remove spurious string copy.

While here fix indentation.

Harbormaster completed remote builds in B75986: Diff 184795.Sun, Aug 23, 3:48 PM

fernape marked an inline comment as done.Sun, Aug 23, 3:49 PM

fernape added inline comments.

usr.bin/whereis/whereis.c
382–383	Thanks for having a look at this. I don't see a reason to duplicate the string, and if at some point I did, I honestly do not remember. While here, let me fix the indentation in the `for` loop.

ngie added a subscriber: ngie.Sat, Aug 29, 3:06 PM

ngie added inline comments.

usr.bin/whereis/whereis.c
323
328–330
334
336–337	This feels unnecessary given the prior stat call.
367–369	Please use asprintf, etc, to build paths instead of 3 separate calls.

Address feedback by ngie@

Harbormaster completed remote builds in B76351: Diff 185486.Mon, Aug 31, 5:05 PM

fernape marked 5 inline comments as done.Mon, Aug 31, 5:07 PM

fernape added inline comments.

usr.bin/whereis/whereis.c
367–369	There are other (old) instances of this sequential triplet of `strcat` calls around. Let me know if you think we should change them as well.

ngie added inline comments.Mon, Aug 31, 5:21 PM

usr.bin/whereis/whereis.c
317	Where is `b` freed after this change?
348–352	Let's get rid of an antique CVS reference while here :).
357–359	This seems a bit suspect: you could technically set the locale to "C" to get this behavior and use `isupper` here. Not sure why the original author thought this was the best approach to take...
367–369	There are other (old) instances of this sequential triplet of `strcat` calls around. Let me know if you think we should change them as well. Sounds like a good junior cleanup task / opportunity for adding new tests (as needed!) :).