Add htcondor-cli package

conda-forge / htcondor-feedstock

A conda-smithy repository for htcondor.

BSD 3-Clause "New" or "Revised" License

0 stars 7 forks source link

Add htcondor-cli package #206

Closed duncanmmacleod closed 10 months ago

duncanmmacleod commented 11 months ago

This PR adds a new htcondor-cli package that bundles the htcondor_cli Python package.

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy (Use the phrase code>@<space/conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

conda-forge-webservices[bot] commented 11 months ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

duncanmmacleod commented 11 months ago

@timtheisen, the OSX build failures are being addressed elsewhere, but can you please review the basic idea, which is:

create a new htcondor-cli conda package that bundles the htcondor-cli Python package
the htcondor-cli Python package is just an installed copy of the htcondor_cli Python library copied out of the src/condor_tools source directory.

I'm not sure if there is a plan to separately distribute the htcondor_cli Python stuff in other distributions; if there is please let me know and I can try to follow any (naming) conventions that are likely to be used elsewhere (e.g. for RPMs).

timtheisen commented 11 months ago

We don't have a plan to separate out the htcondor cli into a sub-package.

LGTM

duncanmmacleod commented 11 months ago

We don't have a plan to separate out the htcondor cli into a sub-package.

@timtheisen, I would recommend folding htcondor_cli into the Python bindings subpackage, as htcondor.cli (similar to htcondor.dags). I posted this as a comment on https://opensciencegrid.atlassian.net/browse/HTCONDOR-2186?focusedCommentId=382564.