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 generic installers #6028

Merged
merged 2 commits into from
Mar 5, 2024
Merged

fix generic installers #6028

merged 2 commits into from
Mar 5, 2024

Conversation

hgy59
Copy link
Contributor

@hgy59 hgy59 commented Mar 4, 2024

Description

This fixes the generic installers that are broken with #5927 for all SRM and DSM < 6.0 packages.

Type of change

  • Includes framework fixes

@hgy59 hgy59 added the framework label Mar 4, 2024
@hgy59
Copy link
Contributor Author

hgy59 commented Mar 4, 2024

@th0ma7, @mreid-tt as commented in #5927

This is a fast fix that prints pip version like:

pip 23.1.2 from /volume1/@appstore/python310/lib/python3.10/site-packages/pip (python 3.10)

The original attempt was to put the pip version into a variable for further processing.

IMO there is no need to log the pip version at all.

@hgy59 hgy59 mentioned this pull request Mar 4, 2024
6 tasks
@hgy59 hgy59 merged commit 0fda56d into SynoCommunity:master Mar 5, 2024
1 check passed
@hgy59 hgy59 deleted the fix_installers branch March 5, 2024 06:11
@mreid-tt
Copy link
Contributor

mreid-tt commented Mar 5, 2024

@hgy59 thanks for you identifying and fixing this. I did a quick filter for builds in the repo uploaded after Jan 27 (when #5927 was merged) with firmware SRM or DSM < 6.0. I found zsh, zsh-static, go2rtc and ruby. These don't seem to be python related packages. Am I correct in thinking that we should not need to re-publish anything as a result of this fix?

@hgy59
Copy link
Contributor Author

hgy59 commented Mar 5, 2024

Am I correct in thinking that we should not need to re-publish anything as a result of this fix?

No, that is not correct.
The installer fails for all packages, unrelated to python, since the functions file is included in the installer script (service-setup).

The error on DSM 5.2 (in /var/log/synopkg.log) was:

/volume1/@tmp/pkginstall/scripts/preupgrade: /volume1/@tmp/pkginstall/scripts/functions: line 75: syntax error: unexpected redirection

It fails when loading the functions file, not when executing a function.

So the answer is yes, we need to recreate and publish affected DSM 5.x and SRM packages.

@hgy59
Copy link
Contributor Author

hgy59 commented Mar 5, 2024

I found zsh, zsh-static, go2rtc and ruby.

I can recreate those packages for SRM and DSM 5.x since I have published those, I only need a make spkclean plus make and must not rebuild the whole packages.

First I will delete the affected packages in the repository.

@hgy59
Copy link
Contributor Author

hgy59 commented Mar 5, 2024

Done: zsh, zsh-static, go2rtc and ruby recreated, published and activated.

@hgy59 hgy59 mentioned this pull request Mar 6, 2024
48 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants