mention exceptions to the EAPI rule
We don't want people updating the EAPI value if they're just importing
a Gentoo ebuild, so note that exception.
BUG=chromium:340036
TEST=tried uploading an EAPI=3 ebuild in chromiumos-overlay
TEST=`./pre-upload_unittest.py` passes
Change-Id: I340f82688b718e5fda63180f045e5b55483a4d2c
Reviewed-on: https://chromium-review.googlesource.com/185063
Reviewed-by: Gilad Arnold <garnold@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
diff --git a/pre-upload.py b/pre-upload.py
index dea41bc..ae78c14 100755
--- a/pre-upload.py
+++ b/pre-upload.py
@@ -498,8 +498,8 @@
A HookFailure or None.
"""
# If this is the portage-stable overlay, then ignore the check. It's rare
- # that we're doing anything other than importing files from upstream, so
- # forcing a rev bump makes no sense.
+ # that we're doing anything other than importing files from upstream, and
+ # we shouldn't be rewriting things fundamentally anyways.
whitelist = (
'chromiumos/overlays/portage-stable',
)
@@ -542,7 +542,9 @@
url = 'http://dev.chromium.org/chromium-os/how-tos-and-troubleshooting/upgrade-ebuild-eapis'
# pylint: enable=C0301
return HookFailure(
- 'These ebuilds are using old EAPIs. Please update to 4 or newer.\n'
+ 'These ebuilds are using old EAPIs. If these are imported from\n'
+ 'Gentoo, then you may ignore and upload once with the --no-verify\n'
+ 'flag. Otherwise, please update to 4 or newer.\n'
'\t%s\n'
'See this guide for more details:\n%s\n' %
('\n\t'.join(['%s: EAPI=%s' % x for x in bad_ebuilds]), url))