Skip to content

*/*: fix variable order#170

Closed
gonsoos wants to merge 12 commits intogentoo:devfrom
gonsoos:dev
Closed

*/*: fix variable order#170
gonsoos wants to merge 12 commits intogentoo:devfrom
gonsoos:dev

Conversation

@gonsoos
Copy link
Contributor

@gonsoos gonsoos commented Apr 27, 2024

I manually went through some entries for pkgcheck scan -c VariableOrderCheck.
I'll keep pushing commits by category, feel free to merge this whenever.

@gonsoos gonsoos closed this Apr 27, 2024
@gonsoos gonsoos reopened this Apr 27, 2024
@gonsoos gonsoos changed the title dev-libs/*: fix variable order */*: fix variable order Apr 28, 2024
@MrRoy
Copy link
Contributor

MrRoy commented Apr 30, 2024

Looks like there are conflicts now, can you check them?

@gonsoos
Copy link
Contributor Author

gonsoos commented May 1, 2024

This should do it. Thanks for the ping!

@gonsoos gonsoos force-pushed the dev branch 2 times, most recently from 2419d20 to 9db7434 Compare May 2, 2024 00:31
@stkw0
Copy link
Contributor

stkw0 commented May 4, 2024

You plan to add any further commits? I guess touching so many files would produce more conflicts if this PR is left for too much time without being merge.

@gonsoos gonsoos force-pushed the dev branch 2 times, most recently from 7efea07 to 702d01f Compare May 8, 2024 02:59
@gonsoos
Copy link
Contributor Author

gonsoos commented May 8, 2024

You plan to add any further commits?

Yes, I want to fix the whole repo. (278 ebuilds to go; this should and could probably be automated)

I guess touching so many files would produce more conflicts if this PR is left for too much time without being merge.

I'd prefer merging these changes in batches, instead of waiting for the rest :')

@gonsoos gonsoos force-pushed the dev branch 4 times, most recently from 976a3a5 to 435449a Compare May 12, 2024 12:18
@MrRoy
Copy link
Contributor

MrRoy commented May 13, 2024

@antecrescent is this ready to merge, then? If not, could you change this PR to a draft until it is?

gonsoos added 11 commits May 13, 2024 18:50
Manually went through some entries for `pkgcheck scan -c
VariableOrderCheck`.

Signed-off-by: Lucio Sauer <watermanpaint@posteo.net>
I manually went through some entries for `pkgcheck scan -c
VariableOrderCheck`.

Signed-off-by: Lucio Sauer <watermanpaint@posteo.net>
I manually went through some entries for `pkgcheck scan -c
VariableOrderCheck`.

Signed-off-by: Lucio Sauer <watermanpaint@posteo.net>
I manually went through some entries for `pkgcheck scan -c
VariableOrderCheck`.

Signed-off-by: Lucio Sauer <watermanpaint@posteo.net>
I manually went through some entries for `pkgcheck scan -c
VariableOrderCheck`.

Signed-off-by: Lucio Sauer <watermanpaint@posteo.net>
I manually went through some entries for `pkgcheck scan -c
VariableOrderCheck`.

Misc updates:
UnusedEclassesCheck
RESTRICT+=" .." -> RESTRICT=".."

Signed-off-by: Lucio Sauer <watermanpaint@posteo.net>
Signed-off-by: Lucio Sauer <watermanpaint@posteo.net>
Signed-off-by: Lucio Sauer <watermanpaint@posteo.net>
Signed-off-by: Lucio Sauer <watermanpaint@posteo.net>
Signed-off-by: Lucio Sauer <watermanpaint@posteo.net>
Signed-off-by: Lucio Sauer <watermanpaint@posteo.net>
Signed-off-by: Lucio Sauer <watermanpaint@posteo.net>
@gonsoos
Copy link
Contributor Author

gonsoos commented May 13, 2024

Is this ready to merge, then?

@MrRoy Yes, please merge it. There might have been a miscommunication..

In an earlier comment (#170 (comment)), I intended to say that I do plan to fix the variable order for all ebuilds in ::guru, but that I will do so in separate PRs to avoid the need for constant rebases and to keep the work of reviewing manageable.

Thank you for keeping an eye on the PR!

@MrRoy
Copy link
Contributor

MrRoy commented May 13, 2024

@antecrescent ok perfect, then I will merge this.
Thanks for your work! :)

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