Skip to content

lib: Split checkMkDerivationFor into multiple functions#17

Merged
jtojnar merged 2 commits into
masterfrom
split-lib
Jan 25, 2021
Merged

lib: Split checkMkDerivationFor into multiple functions#17
jtojnar merged 2 commits into
masterfrom
split-lib

Conversation

@jtojnar

@jtojnar jtojnar commented Jan 25, 2021

Copy link
Copy Markdown
Owner

This will allow us to reuse it for other package sets like Python.

cc @rmcgibbo

This will allow us to reuse it for other package sets like Python.
@rmcgibbo

Copy link
Copy Markdown
Collaborator

This looks good to me. If you want to add the feature of passing the originalDrv to the check function into this PR too, I can rebase #15 on top of this.

This will allow us to do things like checking compatibility of interpreter of the original package.
@jtojnar

jtojnar commented Jan 25, 2021

Copy link
Copy Markdown
Owner Author

Done.

@rmcgibbo

Copy link
Copy Markdown
Collaborator

Cool. Once this is merged, I'll rebase #15.

@jtojnar jtojnar merged commit f9e9674 into master Jan 25, 2021
@jtojnar jtojnar deleted the split-lib branch January 25, 2021 20:27
@jtojnar

jtojnar commented Jan 25, 2021

Copy link
Copy Markdown
Owner Author

Done. Note also the changed name of the drv argument to drvArgs.

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