[arch-projects] [namcap] [PATCH] makedepends: Make VCS matching more robust
Eli Schwartz
eschwartz at archlinux.org
Sun Dec 2 05:04:35 UTC 2018
On 12/1/18 9:54 AM, Michael Straube via arch-projects wrote:
> If a VCS source is renamed using the "::" syntax the makedepends are not
> detected. If there are files starting with <name of VCS binary> in the source
> array false positives are produced. See the gitlab package for example. Make
> the matching more robust to avoid such issues.
>
> Signed-off-by: Michael Straube <michael.straube at posteo.de>
> ---
> Perhaps there is a more elegant way?
>
> Namcap/rules/makedepends.py | 2 +-
> Namcap/tests/pkgbuild/test_makedepends.py | 9 ++++++++-
> 2 files changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/Namcap/rules/makedepends.py b/Namcap/rules/makedepends.py
> index 2a0ceaa..710e969 100644
> --- a/Namcap/rules/makedepends.py
> +++ b/Namcap/rules/makedepends.py
> @@ -56,7 +56,7 @@ class VCSMakedepends(PkgbuildRule):
> missing = []
>
> for v in vcs:
> - if not any(s.startswith(v) for s in pkginfo["source"]):
> + if not any(s.split("::")[-1].startswith(v + '+') for s in pkginfo["source"]):
Instead this fails to detect git:// instead?
The check makepkg uses is to strip ::* and then strip ://* to get the
protocol, and match on protocols like git*, although I have pending
patches to also strip +* and match protocols exactly.
> continue
> d = vcs[v]
> if 'makedepends' not in pkginfo:
> diff --git a/Namcap/tests/pkgbuild/test_makedepends.py b/Namcap/tests/pkgbuild/test_makedepends.py
> index 78c476a..0309016 100644
> --- a/Namcap/tests/pkgbuild/test_makedepends.py
> +++ b/Namcap/tests/pkgbuild/test_makedepends.py
> @@ -77,7 +77,7 @@ makedepends=()
> license=('GPL')
> options=('!libtool')
> source=(bzr+https://ftp.example.com/pub/mypackage
> - git+https://ftp.example.com/pub/mypackage
> + some_name::git+https://ftp.example.com/pub/mypackage
> hg+https://ftp.example.com/pub/mypackage
> svn+https://ftp.example.com/pub/mypackage)
> md5sums=('abcdefabcdef12345678901234567890')
> @@ -104,4 +104,11 @@ package() {
> set(("missing-vcs-makedeps %s", i) for i in makedeps))
> self.assertEqual(r.infos, [])
>
> + def test_example2(self):
> + # Example 2
> + r = self.run_on_pkg(self.pkgbuild1 + 'source=(gitsomething)')
> + self.assertEqual(r.errors, [])
> + self.assertEqual(r.warnings, [])
> + self.assertEqual(r.infos, [])
> +
> # vim: set ts=4 sw=4 noet:
>
--
Eli Schwartz
Bug Wrangler and Trusted User
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.archlinux.org/pipermail/arch-projects/attachments/20181202/b2215a12/attachment.asc>
More information about the arch-projects
mailing list