Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1352169 - Review Request: zerotier-one - Network Virtualization Everywhere https://www.zerotier.com/
Summary: Review Request: zerotier-one - Network Virtualization Everywhere https://www....
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: FE-NEEDSPONSOR FE-DEADREVIEW
TreeView+ depends on / blocked
 
Reported: 2016-07-01 21:53 UTC by Kristof Szabo
Modified: 2020-08-10 00:55 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-08-10 00:55:37 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Kristof Szabo 2016-07-01 21:53:54 UTC
Spec URL: http://lxsystems.de/zero/zerotier.spec
SRPM URL: http://lxsystems.de/zero/zerotier-1.1.4-2.src.rpm
Description: ZeroTier () is a GPLv3 licenced software based network virtualization tool. The software source is maintained on GitHub (https://github.com/zerotier/ZeroTierOne). The current binary installers (distributed on zerotier.com) are built with a self-maintained non-standard build system. The intention of this package is to properly integrate the software into the Fedora/RedHat/CentOS eco-system.
Additional info: this is my first package submission, I hope it doesn't contain huge issues. I'm not sure about how to support old init.d and new systemd based systems in the same time: the %posttrans section of the .spec file contains the respective logic (this is how it is done in the original zerotier rpms as well). I'm almost sure this is not ok, though I have no idea how to manage this in a clean way (I would like to avoid two separate srpms).  
The package builds on koji with "14733246 build (f25, zerotier-1.1.4-2.src.rpm) completed successfully" (http://koji.fedoraproject.org/koji/taskinfo?taskID=14733246)
Fedora Account System Username: kris11

Comment 1 Kristof Szabo 2016-07-01 21:59:05 UTC
I think I should have mentioned explicitly that I'm seeking for a sponsor.

Comment 2 Jonny Heggheim 2016-07-03 10:51:45 UTC
> I'm not sure about how to support old init.d and new systemd based systems in the same time

Upstream solved this in a good way in their spec file: https://github.com/zerotier/ZeroTierOne/blob/master/zerotier-one.spec

Comment 3 Kristof Szabo 2016-07-03 18:21:29 UTC
Thanks for looking at this, upstream just released a new stable version 4 days ago which I overlooked. Let me have a look at this new spec file (whether it builds with koji, etc.) and come back with an update.

Comment 4 François Kooman 2016-07-04 19:16:43 UTC
I've been working on the same thing for the last month and was about to open a review request when I found this one. I've been working with upstream on packaging this and dealing with unbundling the included libraries in the `ext` folder and following the Fedora oackaging guidelines to the best of my knowledge and requesting updates to the various dependencies of the packages in rawhide, f24 and epel.

All dependencies are sorted out now in rawhide and available in updates-testing (F24) and epel-testing (EL7).

SPEC: https://fkooman.fedorapeople.org/zerotier-one/zerotier-one.spec
SRPM: https://fkooman.fedorapeople.org/zerotier-one/zerotier-one-1.1.6-2.fc23.src.rpm

Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=14766141

Upstream is working on builds for other platforms, especially CentOS 6 not using systemd and for customers not wanting to enable EPEL on their machines, that is why there is a spec file in the zerotier repo.

I'm already a Fedora packager and would only require a (formal) review, but am using the `zerotier-one` name as the package name instead of `zerotier`. So not sure if I should open a new request, or we can somehow work together on this?

Comment 5 François Kooman 2016-07-04 19:33:57 UTC
Upstream bug regarding LDFLAGS override that still needs to be fixed: https://github.com/zerotier/ZeroTierOne/issues/346

Comment 6 Kristof Szabo 2016-07-06 21:19:57 UTC
I agree, let's stick to upstream, thanks for the update.

Comment 7 François Kooman 2016-07-07 08:35:01 UTC
EPEL: miniupnpc 2.0 is now available in epel repository, still waiting for json-parser to be moved from epel-testing to epel.

Fedora: miniupnpc 2.0 is not yet pushed to updates for F24, there will be no push to F23.

There are also COPR builds of this, for which currently only the rawhide build works, but the EPEL7 and F24 build should be working shortly if the above is resolved.

https://copr.fedorainfracloud.org/coprs/fkooman/zerotier/

Comment 9 François Kooman 2016-07-11 11:12:58 UTC
Builds on Fedora 24 now as well, not yet on PPC as the tree is not in sync yet.

https://copr.fedorainfracloud.org/coprs/fkooman/zerotier/monitor/

Comment 11 François Kooman 2016-07-15 10:08:24 UTC
The zerotier-one package now also builds on EPEL 7! 

https://copr.fedorainfracloud.org/coprs/fkooman/zerotier/build/385149/

Comment 14 François Kooman 2017-03-18 11:42:54 UTC
SRPM: https://fkooman.fedorapeople.org/zerotier-one/zerotier-one-1.2.2-1.fc25.src.rpm
SPEC: https://fkooman.fedorapeople.org/zerotier-one/zerotier-one.spec

There are some issues with using external libs, a new dependency "json-parser" was added in 1.2.x, I'll try to get the patch upstream for having the ability to use a flag to use the system lib of json-parser, and also http-parser which seemed to have caused issues because distros ship an old version.

Also, the build fails on CentOS 7 due to compiler issues, it seems Clang is needed on CentOS 7 to build ZeroTier.

COPR build: https://copr.fedorainfracloud.org/coprs/fkooman/zerotier/build/528687/

Comment 15 François Kooman 2017-03-18 11:53:18 UTC
The PR for working with system libraries (work in progress): https://github.com/zerotier/ZeroTierOne/pull/462

Comment 16 François Kooman 2017-03-18 12:03:32 UTC
Also: lz4 is now embedded in the source code in a namespace in `node/Packet.cpp`: https://github.com/zerotier/ZeroTierOne/blob/cfe0d0971f3ce5972d955250dc1ff6ec7a30e3f7/node/Packet.cpp

Comment 17 Robert-André Mauchin 🐧 2017-10-02 18:31:59 UTC
 - Please update to version 1.2.4

 - Why do you use a git commit instead of a released version?

Source0:        https://github.com/zerotier/ZeroTierOne/archive/%{version}/%{name}-%{version}.tar.gz

 - make install DESTDIR=$RPM_BUILD_ROOT → %make_install

 - rm -rf $RPM_BUILD_ROOT is not needed in %install

Comment 18 François Kooman 2017-10-02 18:53:08 UTC
I'm no longer interested in packaging ZeroTier due to disagreement with upstream on how to handle library (un)bundling. 

See https://github.com/zerotier/ZeroTierOne/pull/462

It's their choice of course and I respect that, but I just don't want to spend any time fighting upstream :)

Comment 19 Package Review 2020-07-10 00:54:54 UTC
This is an automatic check from review-stats script.

This review request ticket hasn't been updated for some time. We're sorry
it is taking so long. If you're still interested in packaging this software
into Fedora repositories, please respond to this comment clearing the
NEEDINFO flag.

You may want to update the specfile and the src.rpm to the latest version
available and to propose a review swap on Fedora devel mailing list to increase
chances to have your package reviewed. If this is your first package and you
need a sponsor, you may want to post some informal reviews. Read more at
https://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group.

Without any reply, this request will shortly be considered abandoned
and will be closed.
Thank you for your patience.

Comment 20 Package Review 2020-08-10 00:55:37 UTC
This is an automatic action taken by review-stats script.

The ticket submitter failed to clear the NEEDINFO flag in a month.
As per https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews
we consider this ticket as DEADREVIEW and proceed to close it.


Note You need to log in before you can comment on or make changes to this bug.