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

Alpine package registry: Not all possible fields are present in generated APKINDEX.tar.gz #28704

Closed
joniw opened this issue Jan 5, 2024 · 3 comments · Fixed by #28899
Closed

Comments

@joniw
Copy link

joniw commented Jan 5, 2024

Description

Hello,

in the current code for generating the APKINDEX.tar.gz the following supported fields are not created and saved:

  • k: - provider priority, integer (corresponds to provider_priority in PKGINFO), optional
  • i: - install if (corresponds to install_if in PKGINFO, concatenated by spaces into a single line)

It would be nice if these fields can be added. I noticed the problem when installing the salt-minion package and the corresponding startup package (salt-minion-openrc) was not installed.

With regards
Joniw

Gitea Version

1.21

Can you reproduce the bug on the Gitea demo site?

No

Log Gist

No response

Screenshots

No response

Git Version

No response

Operating System

No response

How are you running Gitea?

Test instance in Kubernetes

Database

PostgreSQL

@mintyhippoxyz
Copy link

@joniw Can you manually install the sub package or do I have it even worse?

$ apk add vault vault-openrc
ERROR: unable to select packages:
  vault-openrc (no such package):
    required by: world[vault-openrc

@joniw
Copy link
Author

joniw commented Jan 11, 2024

In my case there is no problem to install the package manually when i workaround #26691.

At this moment i rewrite the architecture of built packages with the workaround described in the issue.

bugaevc added a commit to bugaevc/gitea that referenced this issue Jan 22, 2024
Turn InstallIf into an array as strings (as it has the format as
depend/provides), and add ProviderPriority. Format them into the
geenrated APKINDEX file.

Resolves go-gitea#28704
bugaevc added a commit to bugaevc/gitea that referenced this issue Jan 27, 2024
lunny pushed a commit that referenced this issue Feb 5, 2024
Resolves #28704

Example of an entry in the generated `APKINDEX` file:

```
C:Q1xCO3H9LTTEbhKt9G1alSC87I56c=
P:hello
V:2.12-r1
A:x86_64
T:The GNU Hello program produces a familiar, friendly greeting
U:https://www.gnu.org/software/hello/
L:GPL-3.0-or-later
S:15403
I:36864
o:hello
m:
t:1705934118
D:so:libc.musl-x86_64.so.1
p:cmd:hello=2.12-r1
i:foobar=1.0 !baz
k:42
```

the `i:` and `k:` entries are new.

---------

Co-authored-by: KN4CK3R <admin@oldschoolhack.me>
silverwind pushed a commit to silverwind/gitea that referenced this issue Feb 20, 2024
Resolves go-gitea#28704

Example of an entry in the generated `APKINDEX` file:

```
C:Q1xCO3H9LTTEbhKt9G1alSC87I56c=
P:hello
V:2.12-r1
A:x86_64
T:The GNU Hello program produces a familiar, friendly greeting
U:https://www.gnu.org/software/hello/
L:GPL-3.0-or-later
S:15403
I:36864
o:hello
m:
t:1705934118
D:so:libc.musl-x86_64.so.1
p:cmd:hello=2.12-r1
i:foobar=1.0 !baz
k:42
```

the `i:` and `k:` entries are new.

---------

Co-authored-by: KN4CK3R <admin@oldschoolhack.me>
Copy link

Automatically locked because of our CONTRIBUTING guidelines

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants