-
-
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
Request for SynoCommunity membership #2825
Comments
+1 And sincere thanks, @ymartin59, for being willing to step up and offer your support! |
+1 too |
Additionally, clean-up/merging of all the issues/PR's would really benefit this project and keep it going strong 👍 |
Adding Broadwell to the repo is one thing. That can be done pretty quickly (if it hasn't already) but getting the 6.x toolchain stuff done would be a huge step forward imho! \cc @Diaoul @Dr-Bean For publishing Packages to the repo i think you can go ahead and register yourself an account over here: https://synocommunity.com/register and wait till you get the rights you'll need to publish to the repo ;) EDIT: in fact i've just checked: broadwell is already added as arch to the repo. but since there is no DSM6 Support atm: no packages ;) |
@cytec I already registered with username "ymartin59", so you may grant me rights and provide some information how to proceed with publication Concerning DSM 6.1 toolchain, I am confident, as I already delivered some native packages on demand to requesters |
sorry... you'll have to wait for @Diaoul to give you the rights to publish to the repo. |
Done |
He lives! 💃 |
Thank you but ... what's next ? How should I publish a package properly ? I guess it has to be signed. Is there any documentation about this process ? @Diaoul @KaraokeStu |
@KaraokeStu please reopen, issue is not fixed from my point of view. I am still not member of GitHub organization: https://github.com/orgs/SynoCommunity/people |
@ymartin59 signing is handled by the server. Uploading: https://github.com/SynoCommunity/spksrc/wiki/Package-Server#how-does-it-work-for-developers- thought we had something more clear on that but cant happen to find it now... so to sum this up:
|
Seems like a permission is lacking... POST method is not allowed ?
|
6.X isn't supported yet by the repo... try publishing a 5.2 version of the SPK that should do! |
Yep, I have to work on that. DrBean submited a PR for that. The server will most likely break during any update... |
Oups sorry to learn about that server limitation. As there was already 6.0, I registered 6.1 firmware and tried for a small package easy to test on my platform. May I help on anything ? |
@ymartin59 afaik the server backend doesn't handle 6.x atm as @Diaoul stated there was a PR for @Dr-Bean which has not been implemented yet :/ |
OK so may I implement it ? Where is server source code and installation instruction - for me to setup a testing instance ? What is PR location ? |
AFAIK: the current code can be found here: https://github.com/Diaoul/spkrepo the PR here SynoCommunity/spkrepo#8 |
@cytec @ymartin59 @Dr-Bean @KaraokeStu |
I already thought about it... and PR #2533 for instance refers to a travis build https://travis-ci.org/torrentalle/spksrc/builds/177990202 So a build on Docker image requires to pass a least minimum arguments: package, target architecture. If you have ideas about it, do no hesitate to submit an issue with concept and script proposals, maybe as PR if relevant |
@ymartin59 Bit too busy with my own projects, was just wondering 😉 |
@KaraokeStu @Diaoul @Dr-Bean May you please grant me permissions on SynoCommunity organisation (or else tell me what I should do to deserve it...) ? I would like to close many duplicated "me too" issues and organize job with projects board. There are also ready PRs waiting for merge. Thank you in advance. |
@ymartin59 afaik only @Diaoul can add new members so i guess you'll have to wait till he finds some time again to check back here |
Well, has anyone tried sending him an email? It's listed on his profile. Get all of you full permissions so the project can move forward. |
i've shot him a mail ;) |
Done! Sorry I missed the ping. |
@ymartin59 Do you still want ideas for Travis automation? Since I have been playing around with it a bit. |
@Safihre I saw your configuration file from a PR https://github.com/ymartin59/spksrc/blob/master-travis/.travis.yml I also found some scripts to process PR commits and automatically detect packages to build. |
I fixed the PR #2533. The branch is merged with current |
Hello
I request membership to SynoCommunity organisation so that I can help to triage issues, review and eventually accept merge request, and also publish packages to online repository.
My first target is to address issues with Broadwell architecture and DSM 6.1 support: #2509 #2716 #2718 #2813 #2819 #2666 #2727
As Broadwell will be a new architecture in SynoCommunity repository, packages have to be built for DSM 6.1 only, without doubt about transitioning from 5.x. I agree Broadwell users will do beta-testing but I consider it better than no package being available.
Regards
Yves Martin
The text was updated successfully, but these errors were encountered: