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

builder: Use eopkg.py3 to build pspec.xml recipes #105

Closed
wants to merge 1 commit into from

Conversation

ermo
Copy link
Contributor

@ermo ermo commented Jun 18, 2024

This ensures that solbuild will work with the phase1+2 changes to eopkg and pisi packages here

In addition, make a cosmetic tweak to eopkg install commands (which will effectively be using eopkg.bin when the PR above lands) so the -y flag is set before the -c flag when installing components.

This ensures that solbuild will work with the phase1+2 changes to eopkg
and pisi packages [here](getsolus/packages#2964)

In addition, make a cosmetic tweak to eopkg install commands (which will
effectively be using eopkg.bin when the PR above lands) so the -y flag
is set before the -c flag when installing components.

Signed-off-by: Rune Morling <[email protected]>
@ermo ermo added bug Something isn't working enhancement New feature or request labels Jun 18, 2024
@ermo ermo added this to the 1.7.0 milestone Jun 18, 2024
@ermo ermo marked this pull request as draft June 18, 2024 12:50
@silkeh
Copy link
Member

silkeh commented Sep 18, 2024

FYI: I can't build pisi using eopkg.py3:

 ✓  Now starting build package=pisi
Building source package: pisi
Program terminated.
SyntaxError in COMAR file (/WORK/comar/manager.py): multiple exception types must be parenthesized (error, line 140)
Please use 'eopkg help' for general help.

@silkeh
Copy link
Member

silkeh commented Sep 18, 2024

Closing in favour of #111

@silkeh silkeh closed this Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants