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

Abort when a port includes a nonexistent PortGroup #169

Merged

Conversation

kurthindenburg
Copy link
Contributor

Currently, only a warning is printed and the processing of the port
continues. This typically results in a port not completing later.
After this patch, an error is printed with the port name added and
then the port processing aborts.

closes https//trac.macports.org/ticket/59787

Currently, only a warning is printed and the processing of the port
continues. This typically results in a port not completing later.
After this patch, an error is printed with the port name added and
then the port processing aborts.

closes https//trac.macports.org/ticket/59787
@kurthindenburg
Copy link
Contributor Author

Ideally we would want to check the entire portfile before aborting but that doesn't seem trivial. The return is needed to abort processing when invalid PortGroups are in dependencies.

src/port1.0/portutil.tcl Outdated Show resolved Hide resolved
@kurthindenburg kurthindenburg changed the title Abort when a port includes an invalid PortGroup Abort when a port includes a nonexistent PortGroup Apr 19, 2020
@kurthindenburg
Copy link
Contributor Author

jmr approved on IRC

@kurthindenburg kurthindenburg merged commit d913932 into macports:master Apr 24, 2020
@kurthindenburg kurthindenburg deleted the abort_on_nonexistent_portgroup branch April 24, 2020 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants