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

tests: use stat -c instead of --format for compatibility #627

Merged
merged 1 commit into from
Sep 30, 2024

Conversation

sertonix
Copy link
Contributor

eg. busybox stat only has -c and not --format

Copy link
Collaborator

@smcv smcv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change looks fine. Please add a Signed-off-by to indicate your acceptance of https://developercertificate.org/ as described in https://github.com/containers/common/blob/main/CONTRIBUTING.md#sign-your-prs.

@sertonix
Copy link
Contributor Author

Ok, done

@smcv
Copy link
Collaborator

smcv commented Apr 19, 2024

@alexlarsson, @cgwalters: Please confirm whether you are intending to enforce the rule that Signed-off-by must be from (something that looks like) a real name, and not from an obvious pseudonym?

This PR and #618 are both blocked by having a decision on that. I would personally be fine with a pseudonymous Signed-off-by, especially for small contributions - but I didn't write the rule, so I don't feel that I can be the one to change it.

@smcv smcv requested review from cgwalters and alexlarsson April 19, 2024 16:09
@smcv
Copy link
Collaborator

smcv commented Sep 30, 2024

@alexlarsson, @cgwalters: Please confirm whether you are intending to enforce the rule that Signed-off-by must be from (something that looks like) a real name, and not from an obvious pseudonym?

This PR and #618 are both blocked by having a decision on that.

Right, enough is enough: I've waited too long for a decision on whether this and several other simple fixes can be landed, and it's not like we can tell whether any given contribution is Signed-off-by someone's legal name or a plausible pseudonym anyway, so I'm going to say the answer is "yes" and start merging pseudonymous contributions.

@alexlarsson, @cgwalters, if this is a problem, please revert the relevant changes (and then get equivalent fixes some other way).

@smcv
Copy link
Collaborator

smcv commented Sep 30, 2024

so I'm going to say the answer is "yes" and start merging pseudonymous contributions

(Except not right now, because unrelatedly the Github Actions configuration has regressed, so if anyone has objections to this, you have until the Actions are fixed to raise your objection.)

eg. busybox stat only has -c and not --format

Signed-off-by: Sertonix <[email protected]>
Signed-off-by: Simon McVittie <[email protected]>
@smcv smcv merged commit 2d38286 into containers:main Sep 30, 2024
4 checks passed
@cgwalters
Copy link
Collaborator

It looks like the most canonical docs for this are https://github.com/cncf/foundation/blob/main/dco-guidelines.md

I believe the signoff line here qualifies.

@sertonix sertonix deleted the stat--format branch October 1, 2024 16:33
@sertonix
Copy link
Contributor Author

sertonix commented Oct 1, 2024

so I'm going to say the answer is "yes" and start merging pseudonymous contributions

Thank you!

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