command used:
```
find -type f -name '*.ebuild' -exec sed -i -E 's/^ /\t/g' {} +
```
I was running this command anyway after when I was working on a few
packages because my formatter broke, decided to run it on the root of
the repo then because I was already gonna run it on the packages that I
was working on
I checked each package manually and ran pkgdev to actually see if it was
emitting any warnings, and all of them did for the leading whitepsaces
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
- I did a typo where I used LICENSES instead of LICENSE when trying to
append license for go modules. I fixed that
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
- I did a typo where I used LICENSES instead of LICENSE when trying to
append license for go modules. I fixed that
- go-license was generating Gentoo incompatible license names, the issue
didn't get caught because of the typo. I also fixed that.
See https://bugs.gentoo.org/967017 for more info about the issue related
to go-license
I used my wrapper to fix it, which can be found here:
https://github.com/ingenarel/gentoo-dev-scripts/blob/master/gentoo-go-license
format changes:
- use ${var} instead of $var, which was recommended by negril
- don't use *9999, instead use 9999, it was recommended by negril,
because packages like blender have versions like 5.0.9999, so using
a good format logic is nice
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
- I did a typo where I used LICENSES instead of LICENSE when trying to
append license for go modules. I fixed that
- go-license was generating Gentoo incompatible license names, the issue
didn't get caught because of the typo. I also fixed that.
See https://bugs.gentoo.org/967017 for more info about the issue related
to go-license
I used my wrapper to fix it, which can be found here:
https://github.com/ingenarel/gentoo-dev-scripts/blob/master/gentoo-go-license
format changes:
- use ${var} instead of $var, which was recommended by negril
- don't use *9999, instead use 9999, it was recommended by negril,
because packages like blender have versions like 5.0.9999, so using
a good format logic is nice
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
- modify src_unpack() internally instead of redefining:
functions should be defined on the order that they are called
for more info, check:
https://github.com/gentoo/guru/commit/e19c6d574461655f11cb1c6e1b37ff22c9a6ff70#commitcomment-172006236
- fixed the long line which fixes the following pkgcheck error:
app-misc/watchexec
ExcessiveLineLength: version 2.3.2: excessive line length
(over 120 characters) on line: 8
- used ${var} instead of $var which was recommended by negril
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
- fix redefining src_unpack and instead modify its internals
- update commented out command that gets licenses
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
- eapi 7 -> 8
- disable telemetry
- add necessary upstream info in metadata.xml
- use an ebuild template
- give licenses for dependencies using go-licenses
- add versioned ebuild
- fix shell completions:
previous one wasn't a way to properly install shell completion since
it was trying to just install any .bash file in the root dir
- add zsh completion
- update the minimum required go version
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
I made a mass depfiles conversion to a lot of packages a while ago, and
most of the changes were reverted in
https://github.com/gentoo/guru/pull/354
However, the depfiles in this ebuild went unnoticed by me, so the logic
was fixed: use CRATES instead of depfiles
I also added `RESTRICT="mirror"` which were recommended to me in the IRC
back when the discussion was made on my mass creation of depfiles, this
way, portage shouldn't check for a mirror each time for hundreds of rust
CRATES
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck warnings:
app-misc/pastel
WhitespaceFound: version 9999: ebuild has leading whitespace on lines: 17, 19
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
based on the discussions on https://github.com/gentoo/guru/pull/376 i
made a versioned ebuild, the 9999 ebuild was also changed to create a
proper template that will also work with future releases, if any
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck warnings:
app-office/openoffice-bin
ExcessiveLineLength: version 4.1.15: excessive line length (over 120 characters) on line: 93
Signed-off-by: Joe Kappus <joe@wt.gd>
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
also fixes the following pkgcheck warnings:
app-misc/lf
DeprecatedInsinto: version 27: deprecated insinto usage (use dozshcomp or newzshcomp from shell-completion.eclass instead), line 48: insinto /usr/share/zsh/site-functions
DeprecatedInsinto: version 27: deprecated insinto usage (use dofishcomp or newfishcomp from shell-completion.eclass instead), line 52: insinto /usr/share/fish/vendor_completions.d
DeprecatedInsinto: version 29: deprecated insinto usage (use dozshcomp or newzshcomp from shell-completion.eclass instead), line 48: insinto /usr/share/zsh/site-functions
DeprecatedInsinto: version 29: deprecated insinto usage (use dofishcomp or newfishcomp from shell-completion.eclass instead), line 52: insinto /usr/share/fish/vendor_completions.d
DeprecatedInsinto: version 30: deprecated insinto usage (use dozshcomp or newzshcomp from shell-completion.eclass instead), line 48: insinto /usr/share/zsh/site-functions
DeprecatedInsinto: version 30: deprecated insinto usage (use dofishcomp or newfishcomp from shell-completion.eclass instead), line 52: insinto /usr/share/fish/vendor_completions.d
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
Part-of: https://github.com/gentoo/guru/pull/355
Closes: https://github.com/gentoo/guru/pull/355
Signed-off-by: Lucio Sauer <watermanpaint@posteo.net>
fixes the following pkgcheck warnings:
app-misc/watchexec
TooManyCrates: version 2.1.2: CRATES variable has 440 crates, consider using crates tarball instead
TooManyCrates: version 2.2.1: CRATES variable has 443 crates, consider using crates tarball instead
TooManyCrates: version 2.3.0: CRATES variable has 487 crates, consider using crates tarball instead
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
KEYWORDS are now right below SRC_URI inside the else statement, for
both, live ebuild and versioned ebuild (effectively avoiding to diverge
the live ebuild from the versioned ebuild)
`dosym "$(which tinyxxd)" /usr/bin/xxd` required tinyxxd to be installed
which was an oversight on my part, it should be now fixed because i used
/usr/bin/tinyxxd
also added dev-util/xxd as a blocker to avoid file collisions
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck warnings:
app-vim/rainbow_csv
RedundantLongDescription: metadata.xml longdescription closely matches DESCRIPTION
BadDescription: version 1.10.0_pre20240705: over 80 chars in length
EmptyGlobalAssignment: version 1.10.0_pre20240705: line 18: empty global assignment: IUSE=""
BadDescription: version 9999: over 80 chars in length
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck warnings:
app-shells/zsh-history-substring-search
DeprecatedInsinto: version 1.0.2: deprecated insinto usage (use dozshcomp or newzshcomp from shell-completion.eclass instead), line 24: insinto /usr/share/zsh/site-functions
DeprecatedInsinto: version 1.1.0: deprecated insinto usage (use dozshcomp or newzshcomp from shell-completion.eclass instead), line 24: insinto /usr/share/zsh/site-functions
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pgkcheck warnings:
app-shells/vivid
SuboptimalCratesSeparator: version 0.9.0: line: 8: using - as name-version separator in CRATES is suboptimal, use name@version instead
SuboptimalCratesURICall: version 0.9.0: line: 79: calling '$(cargo_crate_uris)' is suboptimal, use '${CARGO_CRATE_URIS}' for global CRATES instead
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
original package didn't even compile because there were no files/ dir
- added a files dir and the source
- EAPI 7 -> EPAI 8
- add a metadata.xml
also fixes the following pkgcheck errors:
app-portage/showbuild
MissingRemoteId: missing <remote-id type="github">junghans/cj-overlay</remote-id> (inferred from URI 'https://github.com/junghans/cj-overlay')
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the follwoing pkgcheck warnings:
app-shells/zsh-async
DeprecatedInsinto: version 1.8.5_p20211005: deprecated insinto usage (use dozshcomp or newzshcomp from shell-completion.eclass instead), line 40: insinto /usr/share/zsh/site-functions
DeprecatedInsinto: version 1.8.6: deprecated insinto usage (use dozshcomp or newzshcomp from shell-completion.eclass instead), line 38: insinto /usr/share/zsh/site-functions
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck warnings:
app-mobilephone/flashlight
ProvidedEclassInherit: version 0.1.1-r2: line 6: redundant eclass inherit 'xdg-utils', provided by 'gnome2-utils'
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck warnings:
app-misc/tealdeer
SuboptimalCratesSeparator: version 1.6.1-r1: line: 7: using - as name-version separator in CRATES is suboptimal, use name@version instead
SuboptimalCratesURICall: version 1.6.1-r1: line: 192: calling '$(cargo_crate_uris ${CRATES})' is suboptimal, use '${CARGO_CRATE_URIS}' for global CRATES instead
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgchek warnings:
app-misc/walld
StaticSrcUri: version 1.4.4: '1.4.4' in SRC_URI, replace with ${PV}
StaticSrcUri: version 1.4.4: 'walld-1.4.4' in SRC_URI, replace with ${P}
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck warnings:
app-misc/tinyfetch
PkgMetadataXmlIndentation: metadata.xml: metadata.xml has inconsistent indentation on line: 18
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck warnings:
app-misc/kloak
StaticSrcUri: version 0.2.30: '0.2.30' in SRC_URI, replace with ${PV}
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck warnings:
app-misc/proctl
PkgMetadataXmlIndentation: metadata.xml: metadata.xml has inconsistent indentation on lines: 10, 11, 12
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck warnigns:
app-misc/cheat
DeprecatedInsinto: version 4.2.3: deprecated insinto usage (use
dofishcomp or newfishcomp from shell-completion.eclass instead), line
40: insinto /usr/share/fish/vendor_completions.d
DeprecatedInsinto: version 4.2.3: deprecated insinto usage (use
dozshcomp or newzshcomp from shell-completion.eclass instead), line 44:
insinto /usr/share/zsh/site-functions
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck warnings:
app-misc/mxw
MissingRemoteId: missing <remote-id type="github">dkbednarczyk/mxw</remote-id> (inferred from URI 'https://github.com/dkbednarczyk/mxw')
BadFilename: version 0.1.2: bad filename: [ v0.1.2.tar.gz ]
StaticSrcUri: version 0.1.2: '0.1.2' in SRC_URI, replace with ${PV}
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck warnings:
app-misc/ghq
DeprecatedEclassVariable: version 1.1.0: uses deprecated variable on line 9: EGO_SUM (no replacement)
DeprecatedEclassFunction: version 1.1.0: uses deprecated function on line 63: go-module_set_globals (no replacement)
DeprecatedEclassVariable: version 1.1.5: uses deprecated variable on line 8: EGO_SUM (no replacement)
DeprecatedEclassFunction: version 1.1.5: uses deprecated function on line 96: go-module_set_globals (no replacement)
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck warnings:
app-misc/grit
DeprecatedEclassVariable: version 0.3.0: uses deprecated variable on line 8: EGO_SUM (no replacement)
DeprecatedEclassFunction: version 0.3.0: uses deprecated function on line 43: go-module_set_globals (no replacement)
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck warnings:
app-emulation/wslu
PkgMetadataXmlIndentation: metadata.xml: metadata.xml has inconsistent indentation on lines: 9, 10, 11
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck warning:
app-emulation/box86
ExcessiveLineLength: version 0.3.0: excessive line length (over 120 characters) on lines: 24, 25
ExcessiveLineLength: version 0.3.6: excessive line length (over 120 characters) on lines: 24, 25
ExcessiveLineLength: version 9999: excessive line length (over 120 characters) on lines: 23, 24
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck error:
app-admin/sudo-rs
UnknownUseFlags: version 0.2.7: unknown USE flag: 'su'
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck warnings:
app-admin/overmind
DeprecatedEclassVariable: version 2.2.2-r1: uses deprecated variable on line 8: EGO_SUM (no replacement)
DeprecatedEclassFunction: version 2.2.2-r1: uses deprecated function on line 35: go-module_set_globals (no replacement)
also used ego build instead of go build in src_compile()
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck warnings:
app-misc/gopsuinfo
DeprecatedEclassVariable: version 0.1.5: uses deprecated variable on line 8: EGO_SUM (no replacement)
DeprecatedEclassFunction: version 0.1.5: uses deprecated function on line 30: go-module_set_globals (no replacement)
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck warning:
app-emulation/lsfg-vk
ExcessiveLineLength: version 9999: excessive line length (over 120 characters) on lines: 45, 48, 49
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
Fixes the following pkgcheck warning:
app-doc/sway-wiki
MissingRemoteId: missing <remote-id type="github">swaywm/sway</remote-id> (inferred from URI 'https://github.com/swaywm/sway/wiki')
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
fixes the following pkgcheck warning:
app-dicts/dictd-foldoc
StaticSrcUri: version 20180405: '20180405' in SRC_URI, replace with ${PV}
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
This has the following upsides:
- fixes pkgcheck scan's `TooManyCrates` warning.
- becomes more easy to maintain since all it takes is a new vendor file
and bumping the ebuild.
- saves time for users. Portage doesn't try to download 500+ crates from
gentoo mirrors first then trying the crates.io links, instead, it just
downloads one singular vendor file
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
This has the following upsides:
- fixes pkgcheck scan's `TooManyCrates` warning.
- becomes more easy to maintain since all it takes is a new vendor file
and bumping the ebuild.
- saves time for users. Portage doesn't try to download 500+ crates from
gentoo mirrors first then trying the crates.io links, instead, it just
downloads one singular vendor file
Signed-off-by: ingenarel <ingenarelitems@gmail.com>
ingenarel