[Bug 231300] sysutils/py-salt: Fails to start with ZEROMQ=off

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=231300

Ben Woods <woodsb02 at freebsd.org> changed:

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

--- Comment #6 from Ben Woods <woodsb02 at freebsd.org> ---
Created attachment 212238
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=212238&action=edit
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-transport
include:
- zeromq - adds python dependencies for zeromq to the core requirements section
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 section
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 inline
with the selection of the port option.

Thoughts on this patch?

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