Bug 247506

Summary: Example "poudriere ports -c" command example in Porter's Handbook section 10.5.6 errors out if -M directory already populated
Product: Documentation Reporter: Pau Amma <pauamma>
Component: Books & ArticlesAssignee: freebsd-doc (Nobody) <doc>
Status: Closed Overcome By Events    
Severity: Affects Some People CC: bdrewery
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
URL: https://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/testing-poudriere.html

Description Pau Amma 2020-06-23 17:32:28 UTC
Adapting that example to my ports layout (/usr/ports empty and svn co of ports tree under ~/ports for ease of port maintenance) gives me:

% sudo poudriere ports -c -F f none -M ~/ports -p libudisks
[00:00:00] Error: Directory not empty at /home/pauamma/ports

After staring at the poudriere-ports manpage a bit, I tried:
% sudo poudriere ports -c -m null -M ~/ports -p libudisks
[00:00:00] Imported ports tree "libudisks" from /home/pauamma/ports

The rest of the port testing sequence proceeded without incident, so I assume that command was OK, so I'd like to use it instead of the erroring command for the handbook, on the grounds that it's really counterintuitive to have a "directory must only be populated *after* issuing this command" restriction at this point in the workflow implied by the handbook order. However, since I'm not a poudriere expert, I'd like one to comment on the proposed language change before I submit a Phabricator review. (Cc'd bdrewery@FreeBSD.org to that effect, on the grounds that they're the port maintainer and thus presumably know more about poudriere than I do.)
Comment 1 Pau Amma 2020-07-14 08:47:34 UTC
Fixed as part of doc revision 54331 during the July 2020 Bugathon.