| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
https://github.com/void-linux/void-packages/blob/master/Manual.md
states that the description can have a maximum of 72 chars.
|
|
|
|
|
|
|
|
|
|
|
| |
before:
srcpkgs/nawk/template:2: custom variables should use _ prefix: 2
srcpkgs/nawk/template:2: variable set to empty string: 2
after:
srcpkgs/nawk/template:2: custom variables should use _ prefix: xxx=
srcpkgs/nawk/template:2: variable set to empty string: xxx=
Closes: #306 [via git-merge-pr]
|
|
|
|
|
|
|
| |
Some of these are outright wrong (e.g. `echo foo 2>&1`), others are just
more verbose than they need to be.
Closes: #304 [via git-merge-pr]
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
this is painful:
```
distfiles="https://crates.io/api/v1/crates/pkgname/${version}/download>pkgname-${version}.tar.gz"
```
this is better, and xbps-src already supports extracting them:
```
distfiles="https://static.crates.io/crates/pkgname/pkgname-${version}.crate"
```
this replaces void-linux/void-packages#43417 as it adding new `_SITE` variables
has been discouraged
|
| |
|
|
|
|
|
| |
previously, deprecated variables showed some spurious and confusing
messages. now they should have a better message.
|
| |
|
|
|
|
| |
for void-linux/void-packages#40122
|
|
|
|
|
|
| |
fixes #257
Closes: #258 [via git-merge-pr]
|
| |
|
|
|
|
|
|
|
| |
18:59:51 <abby> leah2, can you revert the multiline lint for now, I'm
working on a better way but it's too buggy to stay for now
This reverts commit 16e6e3e9852c988b5bf0b416fabaeaa03588696f.
|
|
|
|
|
|
|
|
|
|
|
| |
- use awk because scan (grep) does not work over multiple lines
- finds the opening and closing of a variable definition, then checks if
it's on the first line and, if not, checks if the first characters are
not <space><:graph:>
- checks both single- and double-quoted strings, in a loop because shell
makes that hard
- detects lines in multi-line strings that end in a backslash
- `| sort -u -k2` so it does not spam, similar to the other indent lint
|
|
|
|
| |
bug introduced in 8c0e21060fd0abdb16c10d62a56ead0a4da75062
|
|
|
|
| |
Closes: #249 [via git-merge-pr]
|
|
|
|
|
|
|
| |
a few lints used the absolute path (`$template`) as prefix instead of
`$argument`
Closes: #246 [via git-merge-pr]
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
before (prints line before make_check):
```
$ xlint R
shlib_provides="libR.so"
/home/abi/void-packages/srcpkgs/R/template:28: explain why the tests fail
```
after:
```
$ xlint R
/home/abi/void-packages/srcpkgs/R/template:28: explain why the tests fail
```
Closes: #245 [via git-merge-pr]
|
|
|
|
| |
Closes: #244 [via git-merge-pr]
|
|
|
|
|
|
|
| |
also add a regex_escape filter to reduce false-positives in a couple
places
Closes: #242 [via git-merge-pr]
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is a slightly naïve solution that does not check if $pkgname
matches the template's directory. That would not be very trivial to do
to work with the git index functionality or subpackages, and I'm not
quite sure how to handle that.
Tested by putting two $pkgname and $version in a template.
Before:
```
$ xlint chezmoi
grep: the -P option only supports a single pattern
chezmoi'
chezmoi: 'pkgname' defined more than once
chezmoi: 'version' defined more than once
chezmoi: Place license= after pkgname=
chezmoi: Place maintainer= after version=
chezmoi:1: Header should be: # Template file for 'chezmoi
chezmoi:4: revision does not appear immediately after version
```
After:
```
$ xlint chezmoi
chezmoi: 'pkgname' defined more than once
chezmoi: 'version' defined more than once
chezmoi: Place license= after pkgname=
chezmoi: Place maintainer= after version=
chezmoi:4: revision does not appear immediately after version
```
|
| |
|
| |
|
|
|
|
| |
Closes: #220 [via git-merge-pr]
|
|
|
|
| |
Closes: #110 [via git-merge-pr]
|
|
|
|
|
|
|
|
| |
In scan() I was using $argument in the sed script, but oftentimes
$arguments is a path including slashes which breaks havoc.
Different way now, hopefully this is correct now. Note that $msg has to
go through sed since some messages use `\2` to show the matching line.
|
|
|
|
|
|
| |
- do not leak $tmpfile in output from scan()
- indent using tabs
- add new options to README
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This commit introduces two new uses of xlint:
$ xlint :PKGNAME
Lints the template corresponding to package PKGNAME as currently
staged in the git index (i.e. the version that would be committed)
$ xlint :
Lints all templates that are changed in the git index.
Useful to run from the pre-commit hook for the void-packages repo.
|
|
|
|
| |
Existing accounts should keep their name.
|
| |
|
| |
|
|
|
|
|
| |
X11 is a derivative of the MIT license and contains the same license
installation requirements.
|
| |
|
|
|
|
| |
Closes: #211 [via git-merge-pr]
|
|
|
|
| |
Closes: #177 [via git-merge-pr]
|
|
|
|
|
|
|
| |
See
https://github.com/void-linux/void-packages/commit/2a748e7e6d9727a7e921d6b9a163e0eb17be98b9
Closes: #205 [via git-merge-pr]
|
| |
|
| |
|
|
|
|
|
|
|
| |
They're introduced by e0979275b4 (11-pkglint-elf-in-usrshare: allow
explicit setting of exceptions, 2020-07-16)
Closes: #195 [via git-merge-pr]
|
|
|
|
|
|
|
| |
These commands were outputting error messages from xdistdir when outside
a void-packages folder, even though the command is supposed to fail silently.
Closes: #190 [via git-merge-pr]
|
|
|
|
| |
Makes it possible to call xlint as: `xlint 0ad`.
|
|
|
|
|
|
| |
Fixes false positive on desc_option_nonfree=
The deprecated nonfree=yes was removed from all templates so this check
is not needed anymore
|
| |
|
|
|
|
| |
Closes: #179 [via git-merge-pr]
|
|
|
|
| |
This reverts commit 124b3b3ae9e42582abcd85ab30fb1a3c3f894632.
|
|
|
|
| |
Closes: #178 [via git-merge-pr]
|
|
|
|
| |
Closes: #173 [via git-merge-pr]
|
|
|
|
|
|
|
|
|
| |
Some packages likes Carla shippes python script in custom location,
xbps-src hasn't been taught to detect it.
Allow pycompile_module for those packages.
Closes: #166 [via git-merge-pr]
|
| |
|
|
|
|
| |
Closes: #146 [via git-merge-pr]
|
| |
|