public inbox for gentoo-portage-dev@lists.gentoo.org
 help / color / mirror / Atom feed
* [gentoo-portage-dev] [PATCH] Switch to new git '# $Id$' header format
@ 2015-08-09 18:11 Sergei Trofimovich
  2015-08-10  7:26 ` Alexander Berntsen
  2015-08-10 14:46 ` Michał Górny
  0 siblings, 2 replies; 3+ messages in thread
From: Sergei Trofimovich @ 2015-08-09 18:11 UTC (permalink / raw
  To: gentoo-portage-dev; +Cc: Sergei Trofimovich

From: Sergei Trofimovich <siarheit@google.com>

Currently repoman complains on every ebuild in ::gentoo.

Signed-off-by: Sergei Trofimovich <siarheit@google.com>
---
 pym/repoman/checks.py | 7 +++----
 pym/repoman/errors.py | 2 +-
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/pym/repoman/checks.py b/pym/repoman/checks.py
index 5f37648..4ab23d1 100644
--- a/pym/repoman/checks.py
+++ b/pym/repoman/checks.py
@@ -81,7 +81,7 @@ class EbuildHeader(LineCheck):
 	# Why a regex here, use a string match
 	# gentoo_license = re.compile(r'^# Distributed under the terms of the GNU General Public License v2$')
 	gentoo_license = '# Distributed under the terms of the GNU General Public License v2'
-	cvs_header = re.compile(r'^# \$Header: .*\$$')
+	id_header = '# $Id$'
 	ignore_comment = False
 
 	def new(self, pkg):
@@ -100,9 +100,8 @@ class EbuildHeader(LineCheck):
 				return errors.COPYRIGHT_ERROR
 		elif num == 1 and line.rstrip('\n') != self.gentoo_license:
 			return errors.LICENSE_ERROR
-		elif num == 2:
-			if not self.cvs_header.match(line):
-				return errors.CVS_HEADER_ERROR
+		elif num == 2 and line.rstrip('\n') != self.id_header:
+			return errors.ID_HEADER_ERROR
 
 
 class EbuildWhitespace(LineCheck):
diff --git a/pym/repoman/errors.py b/pym/repoman/errors.py
index 3833be6..74a5959 100644
--- a/pym/repoman/errors.py
+++ b/pym/repoman/errors.py
@@ -6,7 +6,7 @@ from __future__ import unicode_literals
 
 COPYRIGHT_ERROR = 'Invalid Gentoo Copyright on line: %d'
 LICENSE_ERROR = 'Invalid Gentoo/GPL License on line: %d'
-CVS_HEADER_ERROR = 'Malformed CVS Header on line: %d'
+ID_HEADER_ERROR = 'Malformed Id header on line: %d'
 LEADING_SPACES_ERROR = 'Ebuild contains leading spaces on line: %d'
 TRAILING_WHITESPACE_ERROR = 'Trailing whitespace error on line: %d'
 READONLY_ASSIGNMENT_ERROR = 'Ebuild contains assignment to read-only variable on line: %d'
-- 
2.5.0



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [gentoo-portage-dev] [PATCH] Switch to new git '# $Id$' header format
  2015-08-09 18:11 [gentoo-portage-dev] [PATCH] Switch to new git '# $Id$' header format Sergei Trofimovich
@ 2015-08-10  7:26 ` Alexander Berntsen
  2015-08-10 14:46 ` Michał Górny
  1 sibling, 0 replies; 3+ messages in thread
From: Alexander Berntsen @ 2015-08-10  7:26 UTC (permalink / raw
  To: gentoo-portage-dev

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Looks OK to me. Brian -- merge if it's an OK time to include this
patch, with the whole refactor thing going on.

- -- 
Alexander
bernalex@gentoo.org
https://secure.plaimi.net/~alexander
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQIcBAEBCgAGBQJVyFIOAAoJENQqWdRUGk8BSXYQAKpfLGVJdgQwutq2pvVCFSRL
CbRTAIGFshZu21VPT8wFWV5SEDjACiyj/lXEyCD+mwtHFYrMNsOayNa6bhJEfTm9
YUHBvNVcXbSdre5svgTinwvFr84l+w8JE5S7pofj29+t2wrZlDjL0BcOlZAKJQEI
SEbxPqSpfcleyYwSt7wbsoNK5vXVqRxpL+zz2ZuBFvs9j27QM6roCfVakAQxzrjW
XzbS3XdhbOoaeamb3R7CBREvv/5oX++Cd8BhabqjotR+p9lw37N2RnMudh70s+WP
vCUIAwPbhliNokhasJQyVk/1wTSutPfrn2bKRl6x3kICvr4b0+epx0SXtS0YsflQ
vsk3JD7nE1RweXgH4HFcP8UJesmxI1BYgLAOKa126UzZcy2GsoIgjs53YOm4M1BC
4Gr6L+MIMz3O8YA2c6G7icJ+TzRSrxFqEVLRVhlsLHwg7JEOoLNoMMABrQYWBeLq
6lm/Y5uGIAf0cioh/m1WLXyTW3xW8kgiGQ7pT28EQ1aSAyj54Lu62VDctOEA3fz7
caWHrT0/iBbu66E3uDMZ2wFU1M+SyJBlKftcIVGo2VlyElP1fMYd4ShW0zKO2FEz
tirTrMHxg0MU46lRdPiD3I+Jpmz7jTw1XBF9L0G+fErTr4DIoViNdnuoFQIvVk4b
nh/hvxAVCy26GHVIwgMe
=1Nv8
-----END PGP SIGNATURE-----


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [gentoo-portage-dev] [PATCH] Switch to new git '# $Id$' header format
  2015-08-09 18:11 [gentoo-portage-dev] [PATCH] Switch to new git '# $Id$' header format Sergei Trofimovich
  2015-08-10  7:26 ` Alexander Berntsen
@ 2015-08-10 14:46 ` Michał Górny
  1 sibling, 0 replies; 3+ messages in thread
From: Michał Górny @ 2015-08-10 14:46 UTC (permalink / raw
  To: Sergei Trofimovich; +Cc: gentoo-portage-dev, Sergei Trofimovich

[-- Attachment #1: Type: text/plain, Size: 354 bytes --]

Dnia 2015-08-09, o godz. 19:11:57
Sergei Trofimovich <slyfox@gentoo.org> napisał(a):

> From: Sergei Trofimovich <siarheit@google.com>
> 
> Currently repoman complains on every ebuild in ::gentoo.
> 
> Signed-off-by: Sergei Trofimovich <siarheit@google.com>

Pushed now.

-- 
Best regards,
Michał Górny
<http://dev.gentoo.org/~mgorny/>

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 949 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-08-10 14:46 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-09 18:11 [gentoo-portage-dev] [PATCH] Switch to new git '# $Id$' header format Sergei Trofimovich
2015-08-10  7:26 ` Alexander Berntsen
2015-08-10 14:46 ` Michał Górny

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox