Skip to content

honor inline license.text in recipes - #10

Open
jorgeMFS wants to merge 1 commit into
devfrom
add-license-text-field
Open

jorgeMFS wants to merge 1 commit into
devfrom
add-license-text-field

Conversation

@jorgeMFS

@jorgeMFS jorgeMFS commented May 6, 2026

Copy link
Copy Markdown
Collaborator

Adds an optional license.text field to the recipe schema. When a recipe sets it, the hub stores that text verbatim as the LICENSE artifact in the registry instead of fetching one from the upstream repo. This unblocks recipes for tools whose upstreams carry the license in a source-file header rather than a standalone LICENSE file (lh3/wgsim is the immediate motivating case, but the same shape applies to plenty of small bioinformatics utilities).


Merge order

sbom-implementation is the priority branch and rewrites parts of the builders. This PR conflicts with it on hub/builders/builder.py and hub/validate/validate.py. Rebase after it lands.

Also worth a look: this targets dev, which is currently identical to master. If dev is defunct this should be retargeted.

@WildBunnie WildBunnie closed this May 7, 2026
@WildBunnie

WildBunnie commented May 7, 2026

Copy link
Copy Markdown
Collaborator

we merged #7 so I will close this for now

@jorgeMFS

Copy link
Copy Markdown
Collaborator Author

Reopening — PR #7 (which led to closing this) only addresses the filename of an upstream LICENSE file (LICENSE.txt, COPYING, etc.). It does NOT cover repos where there is no
separate license file at all — wgsim (lh3/wgsim) is the immediate motivating case: its MIT license lives only in the header comment of wgsim.c. This PR is genuinely complementary to #7, not a duplicate.
Once merged, wgsim recipe (add-wgsim-recipe branch, already pushed) can be PR'd.

@jorgeMFS jorgeMFS reopened this May 19, 2026
@jorgeMFS
jorgeMFS force-pushed the add-license-text-field branch from 30a0f41 to 9a5b570 Compare May 21, 2026 10:02
@WildBunnie
WildBunnie changed the base branch from master to dev July 24, 2026 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants