Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow wildcards / globs also for multiple subject-paths #133

Open
sschuberth opened this issue Jun 27, 2024 · 3 comments
Open

Allow wildcards / globs also for multiple subject-paths #133

sschuberth opened this issue Jun 27, 2024 · 3 comments

Comments

@sschuberth
Copy link

sschuberth commented Jun 27, 2024

The action does not find these paths, see this error, although the step before does find the same paths, so something seems to be wrong with wildcard / glob expansion and multiple paths.

sschuberth added a commit to oss-review-toolkit/ort that referenced this issue Jun 27, 2024
Work-around [1] and only attest the main artifacts, not the signature
files.

[1]: actions/attest-build-provenance#133

Signed-off-by: Sebastian Schuberth <[email protected]>
sschuberth added a commit to oss-review-toolkit/ort that referenced this issue Jun 27, 2024
Work-around [1] and only attest the main artifacts, not the signature
files.

[1]: actions/attest-build-provenance#133

Signed-off-by: Sebastian Schuberth <[email protected]>
sschuberth added a commit to oss-review-toolkit/ort that referenced this issue Jun 27, 2024
Work-around [1] and only attest the main artifacts, not the signature
files.

[1]: actions/attest-build-provenance#133

Signed-off-by: Sebastian Schuberth <[email protected]>
@nmwael
Copy link

nmwael commented Jun 28, 2024

I have similar problems..

@bdehamer
Copy link
Collaborator

bdehamer commented Jul 2, 2024

Unfortunately, the @actions/glob package we're using for glob support doesn't support curly brackets (for reasons I don't understand, this appears to be a very intentional decision).

I see that there is an open issue asking for curly bracket support to be added -- you might throw a +1 on there to help it get some attention.

@nmwael
Copy link

nmwael commented Jul 3, 2024

for me it was not clear, It might be worth while to mention that the current workaround are one glob search per attestation in the documentation..

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

No branches or pull requests

3 participants