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

Fix gentoo support #284

Merged
merged 3 commits into from
May 24, 2022
Merged

Fix gentoo support #284

merged 3 commits into from
May 24, 2022

Conversation

nullpointerarray
Copy link
Contributor

this patch includes multiple fixes:

  • the local repository no longer gets automatically updated. Other
    package managers dont do this either and the output can include
    something about an error, that isnt actually a problem, which causes
    distrobox-init to fail.
  • there are multiple curl packages for gentoo. net-misc/curl is the web
    request tool.
  • vte-profile doesnt exist for gentoo. The closest package is probably
    vte.
  • some packages compilation will report errors, which get ignored by the
    build tools, so dont matter, but the mention will cause distrobox-init
    to abort. To prevent this --quiet-build is used to supress to messages
    from compilation

this patch includes multiple fixes:
- the local repository no longer gets automatically updated. Other
package managers dont do this either and the output can include
something about an error, that isnt actually a problem, which causes
distrobox-init to fail.
- there are multiple curl packages for gentoo. net-misc/curl is the web
request tool.
- vte-profile doesnt exist for gentoo. The closest package is probably
vte.
- some packages compilation will report errors, which get ignored by the
build tools, so dont matter, but the mention will cause distrobox-init
to abort. To prevent this --quiet-build is used to supress to messages
from compilation
Apparently the ci wants different command line args than described in
CONTRIBUNTING.md. CONTRIBUNTING.md should be updated
@nullpointerarray
Copy link
Contributor Author

CONTRIBUNTING.md recommends using shfmt -s but the ci uses shfmt -s -ci -sr -kp. Should i update CONTRIBUTING.md?

@nullpointerarray nullpointerarray marked this pull request as ready for review May 21, 2022 13:02
@89luca89
Copy link
Owner

CONTRIBUNTING.md recommends using shfmt -s but the ci uses shfmt -s -ci -sr -kp. Should i update CONTRIBUTING.md?

I've updated it 😄

@89luca89
Copy link
Owner

LGTM, thanks for the contribution!

@89luca89 89luca89 merged commit 546a569 into 89luca89:main May 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants