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.

Sign up for GitHub

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

Jump to bottom

repo reorg bikeshed #573

Open
classabbyamp opened this issue Sep 17, 2023 · 4 comments · May be fixed by #574
Open

repo reorg bikeshed #573

classabbyamp opened this issue Sep 17, 2023 · 4 comments · May be fixed by #574

Comments

@classabbyamp
Copy link
Member

classabbyamp commented Sep 17, 2023 •
edited
Loading

related to: void-linux/void-infrastructure#142

@the-maldridge expressed interest in the new layout being .../<XBPS_ARCH>/..., and I think I agree that this would make the most sense.

I think the best way to implement this in xbps (and having the side benefit of preventing needing to -R other arch's repos when installing something for another XBPS_ARCH) would be to allow templating repo urls:

repository=https://repo-default.voidlinux.org/current/{{XBPS_ARCH}}/
  1. What delimiter characters should be used to template this? {{XBPS_ARCH}}, %XBPS_ARCH% were thoughts I had.
  2. should other templated values be allowed, like ARCH and LIBC?
  3. Could this also be an opportunity to change <arch> -> <arch>-glibc? <arch> would have to stick around as a legacy alias of course.

I am interested in implementing this once some of the details are hashed out
@the-maldridge
Copy link
Member

Very interested in point 3. We could remove a lot of special casing from a lot of places if we could uniformly expect the same identifiers across all targets.

OpenBSD did %p for a long time iirc for their equivalent of this feature. Maybe something similar with a shorter component?

Would is also be worth in the same change getting rid of the specialness of the main repository and putting it into its own directory? I think it would be nicer that way since it means you can have more selective rsync, but it also makes it much more complicated for downstream repos to gracefully cut over. This may just be a change we have to have a flag day for.

@classabbyamp
Copy link
Member Author

+1 to having .../<xbps_arch>/main, but that shouldn't need anything in xbps (except default config files). aside from rsync implications, it would probably make browsing the repo manually more sane (no "too many files" message or something)

@0x5c
Copy link

0x5c commented Sep 18, 2023

This may just be a change we have to have a flag day for.

That would certainly be a way of simplifying void-linux/void-infrastructure#142 since the nginx replacement rules could just be dropped.

@classabbyamp
Copy link
Member Author

classabbyamp commented Sep 18, 2023 •
edited
Loading

I think it would still be valuable to have the redirect rules, to ensure that older installs aren't completely screwed

@classabbyamp classabbyamp linked a pull request Oct 9, 2023 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

lib: add ability to template repo urls with {arch} classabbyamp/xbps
3 participants
@the-maldridge @classabbyamp @0x5c