-
Notifications
You must be signed in to change notification settings - Fork 97
Getting termux-tools
ready for apt
3.0.0
#165
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
base: master
Are you sure you want to change the base?
Conversation
I tested this, and I think it is working for me. There is something strange-looking about the indentation in some places, but I think you might already have noticed, and will be able to resolve that. |
Haven't noticed anything of the sort. |
Oh ok, sure, here is one of them |
That part was already lined up with the line above, it just used spaces instead of tabs for indentation, which doesn't match the rest of the script. |
924edb6
to
8caa682
Compare
There's snapped it back to 3 tabs. |
The problem is also visible in your screenshot, but then you fixed it in a force push so I guess you are now able to see the problem with how it was before |
Then I'm not seeing what you meant. |
I screenshotted your screenshot and drew a line on it, in my opinion the line should be straight BUT it is ok and does not matter very much because:
|
It was aligned with |
Just finished testing the |
Apt 3.0.0 makes the "multiline"
deb822
format the standard for package sources.This PR aims to introduce support for this repo source format in
termux-tools
.This initial commit only adds support in
pkg
.I plan to add support in
termux-info
tomorrow, however that turned into half a refactor and I'm tired.The changes to
pkg
introduce two possible return values fromhas_repo
.deb822
, the new standard format (determined by the.sources
file extension)legacy
, the old oneline format (determined by the.list
file extension)Support for the legacy oneline
.list
format isn't slated to be removed until 20291,so it is kept for backwards compatibility in the meantime.
""
) indicates that the repo is not present in either format.I also fixed a couple shellcheck warnings and some indentation.
I may do a more extensive refactor of
pkg
as part of this PR later.Footnotes
https://man.archlinux.org/man/sources.list.5#ONE-LINE-STYLE_FORMAT ↩