Skip to content

Conversation

@marc-hb
Copy link
Collaborator

@marc-hb marc-hb commented Mar 28, 2020

Successfully tested with both Make and Ninja on Linux.

Non-portable but approved by @jajanusz in former PR #2626

With this commit:

$                  make -C tools/build_tools/ | wc -l
    770

$ VERBOSE=anything make -C tools/build_tools/ | wc -l
  10387

Signed-off-by: Marc Herbert marc.herbert@intel.com

Successfully tested with both Make and Ninja on Linux.

Non-portable but approved by @jajanusz in former PR thesofproject#2626

With this commit:

$                  make -C tools/build_tools/ | wc -l
    770

$ VERBOSE=anything make -C tools/build_tools/ | wc -l
  10387

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
@marc-hb marc-hb marked this pull request as ready for review March 28, 2020 05:41
@marc-hb marc-hb requested a review from jajanusz as a code owner March 28, 2020 05:41
@marc-hb
Copy link
Collaborator Author

marc-hb commented Mar 30, 2020

Tagging @crazoes, I cannot add you as a reviewer for some unknown github reason

Copy link
Contributor

@xiulipan xiulipan left a comment

Choose a reason for hiding this comment

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

thanks for the fix.

@lgirdwood
Copy link
Member

Jenkins known issues.

@lgirdwood lgirdwood merged commit aa6c0f2 into thesofproject:master Mar 30, 2020
@jajanusz
Copy link
Contributor

Tagging @crazoes, I cannot add you as a reviewer for some unknown github reason

When users are marked as Contributor, you cannot assign them. You can assign users with status Member. I think only @lgirdwood can add members, so you need to contact him.

@marc-hb
Copy link
Collaborator Author

marc-hb commented Mar 31, 2020

When users are marked as Contributor, you cannot assign them. You can assign users with status Member

You sound like the first person I met who understand github's permissions :-) Have you found any other documentation than the pages below? They don't define Contributor or Member roles and they don't seem to mention who can be a Reviewer either...

https://help.github.com/en/github/setting-up-and-managing-organizations-and-teams/repository-permission-levels-for-an-organization
https://help.github.com/en/github/setting-up-and-managing-organizations-and-teams/permission-levels-for-an-organization
https://help.github.com/en/github/setting-up-and-managing-your-github-user-account/permission-levels-for-a-user-account-repository

@paulstelian97
Copy link
Collaborator

@marc-hb thesofproject has is an organization in which we can be members, that will give the Member role. I for example am in the sof-developers team of this project, which has write access to this repo and the others from thesofproject. This means I can edit your comments, give reviews that count towards mergeability etc -- everything on the Write permission.

Member simply means that I'm a member of the organization, probably this means member of one of its teams.

I think someone needs to add you to the sof-developers team in order to be able to review.

@marc-hb
Copy link
Collaborator Author

marc-hb commented Apr 1, 2020

I think someone needs to add you to the sof-developers team in order to be able to review.

Do you mean adding not me but @crazoes to sof-developers because only sof-developers can be in the reviewers in the top-right corner of a PR?
I can already add (other) reviewers, I've been doing it for some time now.

@paulstelian97
Copy link
Collaborator

I think someone needs to add you to the sof-developers team in order to be able to review.

Do you mean adding not me but @crazoes to sof-developers because only sof-developers can be in the reviewers in the top-right corner of a PR?
I can already add (other) reviewers, I've been doing it for some time now.

I did mean her, guess my wording was kinda ambiguous.

@marc-hb marc-hb deleted the quiet-alsatplg branch April 3, 2020 00:15
@marc-hb
Copy link
Collaborator Author

marc-hb commented Apr 4, 2020

PR #2710 fixes ./scripts/build-tools.sh -t -f too.

@jajanusz
Copy link
Contributor

jajanusz commented Apr 7, 2020

When users are marked as Contributor, you cannot assign them. You can assign users with status Member

You sound like the first person I met who understand github's permissions :-) Have you found any other documentation than the pages below? They don't define Contributor or Member roles and they don't seem to mention who can be a Reviewer either...

https://help.github.com/en/github/setting-up-and-managing-organizations-and-teams/repository-permission-levels-for-an-organization
https://help.github.com/en/github/setting-up-and-managing-organizations-and-teams/permission-levels-for-an-organization
https://help.github.com/en/github/setting-up-and-managing-your-github-user-account/permission-levels-for-a-user-account-repository

Github doc is generally ok, but it's still lacking most of thing. I just figured out this stuff, by looking how it works ;p
The links that you pasted apply for people that are already in organizations.

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.

5 participants