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

[QA] client linux download repo ships strange PREINSTALL.sh file #9830

Closed
2 tasks done
jnweiger opened this issue Jun 29, 2022 · 2 comments
Closed
2 tasks done

[QA] client linux download repo ships strange PREINSTALL.sh file #9830

jnweiger opened this issue Jun 29, 2022 · 2 comments
Assignees
Labels
Milestone

Comments

@jnweiger
Copy link
Contributor

jnweiger commented Jun 29, 2022

Pre-submission Checks

  • I checked for similar issues, but could not find any. I also checked the closed issues. I could not contribute additional information to any existing issue.
  • I will take the time to fill in all the required fields. I know that the bug report may be dismissed otherwise due to lack of information.

Describe the bug

The *-linux.tar archives generated by our build system include a PREINSTALL.sh with this line

curl --insecure -L https://github.com/sgreben/http-file-server/releases/download/1.3.3/http-file-server_1.3.3_linux_x86_64.tar.gz | tar xz

I see no connection to our code or to the Linux packages in general. Please remove or document the purpose in the script.

There is also an INSTALL.sh which is supposed to automate the manual copy paste instructions. This should be retained.

Expected behavior

We don't recommend any undocumented scripts to end users or admins.

Steps to reproduce the issue

E.g. seen in http://client-linux-install.jw-qa.owncloud.works/89ec4d44d9ec5cae37f53dfea3a382a2/CentOS_7/

Screenshots

No response

Logs

No response

Client version number

2.10.0

Desktop environment (Linux only)

No response

Client package version and origin (Linux only)

No response

Installation path (Windows only)

No response

Server information

No response

Additional context

No response

@TheOneRing TheOneRing added this to the 3.0 milestone Jun 30, 2022
@fmoc
Copy link
Contributor

fmoc commented Jul 5, 2022

Will have a look. This makes absolutely no sense.

@fmoc
Copy link
Contributor

fmoc commented Jul 5, 2022

The line has been included for a couple of years. The PREINSTALL.sh file is primarily used for openSUSE. The http-file-server line was added in April 2019, I can't tell why. I guess it was used for debugging back then. I do not see any reason to keep this in our repositories. I'll remove it from the scripts, future builds will no longer contain them.

@fmoc fmoc closed this as completed Jul 5, 2022
@jnweiger jnweiger mentioned this issue Jul 12, 2022
56 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants