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

docker cross compilation, packaging and testing #129

Open
wants to merge 44 commits into
base: master
Choose a base branch
from

Conversation

lanmarc77
Copy link

No description provided.

@Gustl22 Gustl22 self-requested a review October 9, 2023 08:42
@Gustl22
Copy link
Contributor

Gustl22 commented Feb 7, 2024

Sorry first I haven't had time for a review and also somehow forgot it 😅
I saw the maintainers now provide a build script:
https://github.com/haiwen/seafile-docker/tree/master/build

I will give feedback if they plan to also build for arm.

@lanmarc77
Copy link
Author

The script from the devs seems to go the direction we have went. It does not implement different architectures and platforms though. It assumes Ubuntu and their package naming. But most likely could be run in any architecture environment where Ubuntu is supported.
It would be great if they add multi architecture support and automated pipeline building. Everyone could just fork the repo and click build.
I am on the other hand are in process of moving to a different file syncing solution and will not invest any time in Seafile anymore. Please use all the code especially for the auto build in pipeline and the following tests to automate as much as possible to work for different architectures. RISC V being the next already showing on the horizon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants