Bug 261445

Summary:	app-portage/repoman: static code fixing
Product:	Portage Development	Reporter:	Justin Lecher (RETIRED) <jlec>
Component:	Repoman	Assignee:	Portage team <dev-portage>
Status:	RESOLVED WONTFIX
Severity:	enhancement	CC:	mgorny, zmedico
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---

Description Justin Lecher (RETIRED) gentoo-dev

2009-03-06 12:48:35 UTC

I would like to have some basic static code fixing features for repoman:

1. correct date in header like echangelog does it in ChangeLog
2. sort USE flag alphabetically
(3. sort DEPs might not be wanted)
4. fixing leading an trailing whitespaces


Those came into my mind quickly and I think they could be implemented easily. I would do testing if some does write the code.

Comment 1 Justin Lecher (RETIRED) gentoo-dev

2009-03-06 12:51:45 UTC

5. usedeps

Comment 2 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev

2009-03-06 13:01:53 UTC

Let's see what dev-portage team thinks about this.

Comment 3 Justin Lecher (RETIRED) gentoo-dev

2009-03-06 13:16:57 UTC

5. usedeps

Comment 4 Brian Dolbec (RETIRED) gentoo-dev

2017-03-16 01:39:54 UTC

Current repoman releases do:

1) fix the copyright date header


It loads the use flags into a python variable, so could perform a sort and save

But I'm not as sure about the whitespace, not from detecting/fixing it, but more from the possibility of messing with other intended whitespace and/or specific formatting in other variables.

Comment 5 Sam James archtester

2022-07-12 03:18:23 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.