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.

Sign up for GitHub

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

Jump to bottom

add /bin/sh dependency #245

Closed
wants to merge 1 commit into from
Closed

add /bin/sh dependency #245

wants to merge 1 commit into from

Conversation

shawnbutts
Copy link

Fixes #244

@shawnbutts shawnbutts requested a review from a team as a code owner November 30, 2022 00:13
@linux-foundation-easycla
Copy link

CLA Not Signed

@shawnbutts shawnbutts closed this Nov 30, 2022
@jpkrohling
Copy link
Member

This was looking promising, any reasons for closing the PR?

@shawnbutts
Copy link
Author

The CLA and the private information requirement.

@elmerfud
Copy link

Seems a bit silly to require a personal address to submit a PR.

@jpkrohling
Copy link
Member

jpkrohling commented Nov 30, 2022 •
edited

I agree with you. We are in the process of evaluating a change to have DCO as an alternative to CLA.

edit: thanks @bogdandrutu for the clarification

@shawnbutts
Copy link
Author

In the meantime, maybe someone that has already signed the CLA can get this done.

@jpkrohling jpkrohling mentioned this pull request Nov 30, 2022
Open
@bogdandrutu
Copy link
Member

I agree with you. We are in the process of changing that to have DCO as an alternative to CLA.

We are evaluating, not in a process of changing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Aneurysm9 Aneurysm9 Awaiting requested review from Aneurysm9 Aneurysm9 is a code owner automatically assigned from open-telemetry/collector-contrib-approvers

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

RPM packages are missing a /bin/sh dependency
4 participants
@shawnbutts @jpkrohling @elmerfud @bogdandrutu