-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
subversion: initial package #5296
base: master
Are you sure you want to change the base?
Conversation
8588bef
to
b56ab55
Compare
Hey! Any progress on this? I'm waiting for it to finally be able to update my DSM :D |
This PR depends on #5649. And I will add a dedicated web ui package (probably usvn) |
Any progress? I can see the PR you were depending on has been merged. |
- update cross/apr to v1.7.2 - update cross/apr-util to v1.6.3 - configure subversion to use serf
- use alternate port 3691 to avoid conflicts with SVN package from Synology - use service-variables file to configure shared folder at installation time - add wizard to define shared folder for repositories
- update serf for openssl3 compatibility
- update cyrus-sasl and depend on openssl3
I did some updates now, but see and issue on aarch64
I have to validate other archs... @matt-hall-zory If you don't mind, you could manually install the package built by github build action. |
- update mutt and depend on openssl3
Almost one more year later without SVN on Diskstation :( Can we please get this much needWanted package going? IMHO there is no need for a Web UI integrated in this package. I'd actually prefer dedicated server and web ui packages as they would be easier to maintain individually. Pretty please? :) |
This reverts commit d5c6d06.
This reverts commit 85c48b6.
- update subversion to v1.14.4 - update cross/apr to v1.7.5
@cpm1 do you have some time to test the package created by the github build action? just navigate to the build action and under artifacts download the zip archive for your model (arch). EDIT: |
- use archive url for download (remains valid after upstream updates)
@hgy59 Great! I'll install it, give it a spin and report back in one or two days. Edit: sorry, I'm too dumb. All I can find as artifacts are zip files with the content of the spk's, but no ready to install spk. Where did I turn the wrong way? |
If you expand the zip file and don't get an SPK it's because your decompression software is also decompressing the SPK file. You can check your settings or use another decompression software that won't try to also decompress the SPK file. |
Oops! Of course, you are right as am I with being dumb xD Extracted properly now, installed and first simple tests were successfull. Thank you very much! Just the custom port is a bit irritating. It really should stay the default port at least on DSM7+. Or even better, allow the port to be specified at install time and/or changed via settings later. |
Synology packages typically do not support dynamic port allocation, as the port needs to be explicitly defined during the build process: spksrc/spk/subversion/Makefile Line 17 in 8db6b95
For Subversion, we could use the default port (3690). If the port is already in use on DSM, the system will detect the conflict and block the installation. To address this, we could adopt an approach where DSM 6 uses a custom port while DSM 7 retains the default port, reducing the likelihood of conflicts. @hgy59, what are your thoughts on sticking with the default port? Since SVN is a third-party package, using the same port would align with our approach for Jellyfin, which shares a port with the third-party app Emby. Alternatively, we could implement a version-dependent port allocation, similar to the strategy used for MariaDB. |
Description
Synology does not support subversion on DSM 7, so there are some requests for an svn package.
Checklist
all-supported
completed successfullyType of change