Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 08 Mar 2020 00:55:11 +0000
From:      bugzilla-noreply@freebsd.org
To:        python@FreeBSD.org
Subject:   [Bug 231300] sysutils/py-salt: Fails to start with ZEROMQ=off
Message-ID:  <bug-231300-21822-rMb4PJNNiy@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-231300-21822@https.bugs.freebsd.org/bugzilla/>
References:  <bug-231300-21822@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D231300

Ben Woods <woodsb02@freebsd.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|ports-bugs@FreeBSD.org      |woodsb02@freebsd.org
                 CC|                            |woodsb02@freebsd.org
 Attachment #212238|                            |maintainer-approval?(christ
              Flags|                            |er.edwards@gmail.com)

--- Comment #6 from Ben Woods <woodsb02@freebsd.org> ---
Created attachment 212238
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=3D212238&action=
=3Dedit
Patch to fix python dependencies when ZEROMQ option disabled

There is a build time option --salt-transport that should be set to change
python dependencies specified in the requires.txt. For those not familiar, =
the
structure of the requires.txt file is outlined in the following link:
https://setuptools.readthedocs.io/en/latest/formats.html#requires-txt

According to the salt setup.py code, the supported values for --salt-transp=
ort
include:
- zeromq - adds python dependencies for zeromq to the core requirements sec=
tion
of requires.txt
- raet - adds python dependencies for raet to the core requirements section=
 of
requires.txt
- both - no impact
- ssh - no impact
- none - no impact

https://github.com/saltstack/salt/blob/v2019.2.3/setup.py#L1006

Note that the raet dependencies are already included extra requirements sec=
tion
of requires.txt under the [RAET] optional feature.

It is also worth noting that raet support is removed in the next release of
salt (v3000).

Given this, the attached patch only toggles the zeromq option on or off inl=
ine
with the selection of the port option.

Thoughts on this patch?

--=20
You are receiving this mail because:
You are on the CC list for the bug.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-231300-21822-rMb4PJNNiy>