-
Notifications
You must be signed in to change notification settings - Fork 161
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
options command generating errors about missing ports tree FLAVOR feature #1083
Comments
I'm having the same trouble, different port with a flavor.
This checkout of the ports tree does support flavors, though:
I do note that I'm not seeing any references to There's supposedly a test for empty Again, I'm sorry if I missed something that requires context to understand, this is the first time I've really looked into the repo. |
The SUBPACKAGES feature is still being worked on, support needs to be added in poudriere before it can be unleashed on the ports tree. But we should be getting it at some point. Looks like the bug was introduced when adding support for SUBPACKAGES and not updating the options.sh script, but looking at it I found nothing obvious about how to fix this. |
I saw it was still in progress, yeah. 🙂. Been over a month and a half without movement, so I just assumed it was going to be a "some day" kind of feature. It definitely feels like a miss when adding support for SUBPACKAGES -- I don't think added support was intended to be breaking. Seems like two things need to be done:
Like you, I can't really tell what the correct way is to go about fixing the way poudriere handles a missing feature. And I'm all the way lost in this shell script testing framework. I'm fine with simple scripts, but as soon as we get to multiple directories and sources, I'm lost in the weeds. Poudriere is a marvel of complex shell scripts that my tiny brain can't hold enough of to make any sense. Anything more than a couple screens worth is too much for me. |
This is in theory already done. Obviously if you ask for something that depends on a feature (like a specific flavor) there is not much to be done, skipping it is correct. Reverting on the unflavored port looks arbitrary. Anyway the only issue at hand is the options subcommand not properly populating I tried to figure it out but could not find a solution. I plan to find some spare time in the near future to try harder. |
I have created a pull request with a patch that works fine for me. You can test it and report back. Find it here: #1087 Thanks! |
I updated poudriere-devel to 3.4.99.20231113 which includes the changes in #1087 and the options command now fails with an error saying the port does not exist for any port I try. It's looking for the port in After adding some debug output, the problem is that I've uploaded a PR with a fix, #1089. |
@ahktenzero Thanks for noticing my mistake, I clearly missed this detail of jail_start. |
https://bapt.nours.eu/patch-fix-options-for-flavors.diff this should fix this without having to spawn a jail by default |
@bapt I've tested it with my common usage patterns and it looks fine, does what is expected. |
Fixed by 9143ab4 |
Prerequisites
Describe the bug
With latest
ports-mgmt/poudriere-devel
port running thepoudriere options
command passinga port with a flavor (f.e.net/unison@nox11
) causes errors like:And the port is skipped.
How to reproduce
see above
Expected behavior
No errors being output. No skipping
Environment
Additional context
After further investigation, when running the options command, the P_PORTS_FEATURES variable is not being populated via the options command.
If I get more insight to share I'll do.
The text was updated successfully, but these errors were encountered: