Skip to content

Add pushFilter to allow filtering cache derivations#77

Merged
domenkozar merged 1 commit intocachix:masterfrom
saschagrunert:push-filter
Feb 18, 2021
Merged

Add pushFilter to allow filtering cache derivations#77
domenkozar merged 1 commit intocachix:masterfrom
saschagrunert:push-filter

Conversation

@saschagrunert
Copy link
Copy Markdown
Contributor

This allows filtering derivations to be pushed via an optional
pushFilter variable.

The use case behind it is to not push the actually built derivation, but
only runtime and build dependencies.

@saschagrunert
Copy link
Copy Markdown
Contributor Author

@domenkozar does this have to be more flexible to fix #75 ?

@domenkozar
Copy link
Copy Markdown
Member

@domenkozar does this have to be more flexible to fix #75 ?

I think it's good enough. I'd add two things to the docs:

  • an example, let's say (-source$|nixpkgs\.tar\.gz$)
  • a warning that the filter doesn't guarantee it won't get pushed in case the path is part of the closure of something that will get pushed

@saschagrunert
Copy link
Copy Markdown
Contributor Author

Changed as suggested. 👍

This allows filtering derivations to be pushed via an optional
`pushFilter` variable.

The use case behind it is to not push the actually built derivation, but
only runtime and build dependencies.

Signed-off-by: Sascha Grunert <mail@saschagrunert.de>
@domenkozar domenkozar merged commit 565940c into cachix:master Feb 18, 2021
@saschagrunert saschagrunert deleted the push-filter branch February 18, 2021 15:57
@saschagrunert
Copy link
Copy Markdown
Contributor Author

saschagrunert commented Mar 2, 2021

Hey @domenkozar, do you have planned a new release in the near future? :)

@domenkozar
Copy link
Copy Markdown
Member

I'll try to do it this week.

@saschagrunert
Copy link
Copy Markdown
Contributor Author

I'll try to do it this week.

Thank you so much!

@hmenke
Copy link
Copy Markdown

hmenke commented Mar 29, 2021

Is the next release going to be published soon?

@domenkozar
Copy link
Copy Markdown
Member

Hey @hmenke, scheduled for Thursday.

@domenkozar
Copy link
Copy Markdown
Member

https://github.com/cachix/cachix-action/releases/tag/v9

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.

3 participants