Bug 562812

Summary:	app-portage/repoman: repoman should test for variables in HOMEPAGE
Product:	Portage Development	Reporter:	Jonas Stein <jstein>
Component:	Repoman	Assignee:	Portage team <dev-portage>
Status:	RESOLVED WONTFIX
Severity:	normal	CC:	david, jstein, qa
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
URL:	https://devmanual.gentoo.org/ebuild-writing/variables/
See Also:	https://bugs.gentoo.org/show_bug.cgi?id=292394
Whiteboard:
Package list:		Runtime testing required:	---

Description Jonas Stein gentoo-dev

2015-10-11 10:50:40 UTC

devmanual says:
===
Never refer to a variable name in the string; include only raw text. 
===

This should be tested by repoman. As of today there are 180 packages using variables. [1]

Some look like a testcase for a parser:
app-emulation/docker-swarm-0.4.0:HOMEPAGE="https://docs.docker.com/${PN##*-}/"

[1] tested with:
qgrep -NC HOMEPAGE= | grep "\\$" 

Reproducible: Always

Comment 1 Sebastian Pipping gentoo-dev

2015-10-11 14:41:50 UTC

What do you mean by "Some look like a testcase for a parser"?

Comment 2 Jonas Stein gentoo-dev

2015-10-11 15:09:15 UTC

just a comment on "${PN##*-}/" which makes it not trivial to get the URL from the .ebuild file.

Comment 3 Jonas Stein gentoo-dev

2017-08-24 20:56:55 UTC

working example code to check for server status in HOMEPAGE and SRC_URI:
https://github.com/mm1ke/gentoo-scripts

Comment 4 Sam James archtester

2022-07-12 03:18:39 UTC

repoman support has been removed per bug 835013.

Please file a new bug (or, I suppose, reopen this one) if you feel this check is still applicable to pkgcheck and doesn't already exist.

Comment 5 Sam James archtester

2022-07-12 03:23:27 UTC

pkgcheck has ReferenceInMetadataVar (and HomepageInSrcUri).