Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Working on port message #14

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

JDLH
Copy link

@JDLH JDLH commented Apr 14, 2022

Add a message to the output as the restore_ports script starts to install each port. The message is "Working on port:", followed by port name, followed by variant tags if any. This message is nicely set off from the other output, which is prefixed by --->. Thus if a port or its dependency fails to install, it is clearer which underlying port request started the chain which led to the error.

The output now looks like this:
Working on port: p5.28-web-scraper ...
---> Computing dependencies for p5.28-web-scraper
---> Fetching archive for p5.28-web-scraper
...
---> Cleaning p5.28-web-scraper
Working on port: pango x11 + quartz +...
---> Computing dependencies for pango
---> Fetching archive for pango
...

Add a message to the output as the restore_ports script starts to install each port. The message is "Working on port:", followed by port name, followed by variant tags if any. This message is nicely set off from the other output, which is prefixed by `--->`. Thus if a port or its dependency fails to install, it is clearer which underlying port request started the chain which led to the error.

The output now looks like this:
Working on port: p5.28-web-scraper ...
--->  Computing dependencies for p5.28-web-scraper
--->  Fetching archive for p5.28-web-scraper
...
--->  Cleaning p5.28-web-scraper
Working on port: pango x11 + quartz +...
--->  Computing dependencies for pango
--->  Fetching archive for pango
...
Add "Working on port:" message for each port
@cooljeanius
Copy link
Sponsor Contributor

restore_ports.tcl was just updated in #10; can you rebase?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants