https://bugzilla.redhat.com/show_bug.cgi?id=1164378
Bug ID: 1164378 Summary: Review Request: pulse - Open Source Continuous File Synchronization Product: Fedora Version: rawhide Component: Package Review Severity: medium Assignee: nobody@fedoraproject.org Reporter: javier.wilson@gmail.com QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org
Spec URL: http://axioma.guegue.com/rpms/pulse/pulse.spec SRPM URL: http://axioma.guegue.com/rpms/pulse/pulse-0.10.5-1.fc21.src.rpm Description: Pulse (previously Syncthing) replaces proprietary sync and cloud services with something open, trustworthy and distributed. Your data is your data alone and you deserve to choose where it is stored, if it is shared with some third party, and how it's transmitted over the Internet. Fedora Account System Username: axioma
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
Eduardo Echeverria echevemaster@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |177841 (FE-NEEDSPONSOR)
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
--- Comment #1 from Eduardo Echeverria echevemaster@gmail.com --- Hi Javier, a few comments.
- Given that the source comes from Github is advisable use of the Source url packaging guidelines, please see http://fedoraproject.org/wiki/Packaging:SourceURL#Github - Use %global instead %define, %define is deprecated. - Description must not be over 80 columns per line - Citing http://fedoraproject.org/wiki/PackagingDrafts/Go Binary packages which build against this source will set $GOPATH to '%{_datadir}/gocode' ( or '%{gopath}' in golang > 1.2.1-1) - Try to separate the appropiate build method of the file build.sh. It is simpler and clean.
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
--- Comment #2 from Eduardo Echeverria echevemaster@gmail.com --- Any news in this review?
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
Marc Laporte marc@marclaporte.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |marc@marclaporte.com
--- Comment #3 from Marc Laporte marc@marclaporte.com --- "Pulse is the low-level synchronisation engine at the heart of our platform. But it is not a consumer product and never will be. You can still download and use it but we will no longer be supporting it as a separate product. If you want a standalone synchronisation engine with community support, etc., please use Syncthing instead." Source: https://ind.ie/blog/focus/
Related links: https://discourse.syncthing.net/t/rpm-package-for-fedora/610/3 https://copr.fedoraproject.org/coprs/thunderbirdtr/Syncthing_for_Fedora/ http://www.clearfoundation.com/docs/howtos/syncthing http://suite.tiki.org/Syncthing
Thanks!
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
--- Comment #4 from Marc Laporte marc@marclaporte.com --- "a configuration for running syncthing under the "systemd" service manager on Linux both under either a systemd system service or systemd user service." https://github.com/syncthing/syncthing/tree/master/etc/linux-systemd
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
--- Comment #5 from javierwilson javier.wilson@gmail.com --- Sorry for the long long time it took me to continue working on this. I have taken most of Echeverria's observations, using github guidlines, etc. However, I have not been able to make the build process any clearer, nor improve the use of gopath.
Marc, I have renamed to syncthing. I have not yet used (installed) systemd script.
It finally buidls fine for i696, x86_64 and arv7hl. See http://koji.fedoraproject.org/koji/taskinfo?taskID=8912618
New spec and SRPM: http://axioma.guegue.com/rpms/syncthing/syncthing.spec http://axioma.guegue.com/rpms/syncthing/syncthing-0.10.22-1.fc21.src.rpm
Any comments? Any advise?
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
--- Comment #6 from Marc Laporte marc@marclaporte.com --- Thanks Javier!
This is deeply appreciated.
What your thoughts with respect to EPEL 6 & 7? https://github.com/thunderbirdtr/syncthing_rpm/issues/3 https://github.com/thunderbirdtr/syncthing_rpm/commits/master
And for Syncthing-GTK? https://github.com/syncthing/syncthing-gtk
Thanks!
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
--- Comment #7 from Marc Laporte marc@marclaporte.com --- There is some feedback here: https://discourse.syncthing.net/t/progress-on-rpm-package-review-for-inclusi...
Thanks!
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
--- Comment #8 from javierwilson javier.wilson@gmail.com --- New spec, same thing + systemd file ready to be used at system level. http://axioma.guegue.com/rpms/syncthing/systemd/syncthing.spec
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
--- Comment #9 from javierwilson javier.wilson@gmail.com --- Everything is OK according to koji, RPMs for i686, x86_64 and armv7hl http://koji.fedoraproject.org/koji/taskinfo?taskID=8935993
To enable syncthing for user "bob" you just need to:
systemctl enable syncthing@bob.service systemctl start syncthing@bob.service
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
javierwilson javier.wilson@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|Review Request: pulse - |Review Request: syncthing - |Open Source Continuous File |Open Source Continuous File |Synchronization |Synchronization
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
Fabian Affolter mail@fabian-affolter.ch changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |mail@fabian-affolter.ch
--- Comment #10 from Fabian Affolter mail@fabian-affolter.ch --- The latest release is 0.10.26.
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
--- Comment #11 from Marc Laporte marc@marclaporte.com --- One more RPM: https://okay.com.mx/en/linux/rpm-repositories-for-centos-6-and-7.html
yum install syncthing
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
--- Comment #12 from Fabian Affolter mail@fabian-affolter.ch --- Sorry, it seems that I was not clear. I was talking about an updated SPEC file/SRPM for this review request.
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
Levente Farkas lfarkas@lfarkas.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |lfarkas@lfarkas.org
--- Comment #13 from Levente Farkas lfarkas@lfarkas.org --- is there any progress with request?
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
--- Comment #14 from Levente Farkas lfarkas@lfarkas.org --- Created attachment 1028831 --> https://bugzilla.redhat.com/attachment.cgi?id=1028831&action=edit a bit updated spec file
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
Dusty Mabe dustymabe@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |dustymabe@redhat.com
--- Comment #15 from Dusty Mabe dustymabe@redhat.com --- bump. any progress here? Would like to use syncthing in fedora.
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
Fabio Valentini decathorpe@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |decathorpe@gmail.com
--- Comment #16 from Fabio Valentini decathorpe@gmail.com --- I maintain a copr repository with the latest syncthing and friends (decathorpe/syncthing) and would be interested in either picking up this review request (and updating it with the recent version) or creating a clean, new one and close this one as a duplicate.
I am aware that the packages in my copr repo might not be ready for fedora proper yet, and I don't have much experience packaging golang packages, so please bear with me if it takes some time. You can use the current packages nonetheless (they work as intended) - I have quite a few happy users already.
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
--- Comment #17 from Dusty Mabe dustymabe@redhat.com --- Hey Fabio, Yes I saw your copr. I vote that we abandon this review and allow you to open a new one and/or commandeer this one. I say just try to get your spec file in reviewable state and post it here in this review and then we can work with you to try to get anything fixed up that needs to be. Jan Chaloupka (jchaloup@redhat.com) helped me with my first go package review and he would probably help you too.
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
Fabio Valentini decathorpe@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Resolution|--- |DUPLICATE Last Closed| |2017-02-28 13:55:38
--- Comment #18 from Fabio Valentini decathorpe@gmail.com ---
*** This bug has been marked as a duplicate of bug 1427634 ***
https://bugzilla.redhat.com/show_bug.cgi?id=1164378
Fabio Valentini decathorpe@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks|177841 (FE-NEEDSPONSOR) |
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor
package-review@lists.fedoraproject.org