public inbox for gentoo-portage-dev@lists.gentoo.org
 help / color / mirror / Atom feed
* [gentoo-portage-dev] [PATCH] repoman: declare '-x', '--xmlparse' command line options obsolete
@ 2016-05-19 22:45 Göktürk Yüksek
  2016-05-20  4:14 ` [gentoo-portage-dev] " Duncan
  2016-05-20  9:10 ` [gentoo-portage-dev] " Alexander Berntsen
  0 siblings, 2 replies; 3+ messages in thread
From: Göktürk Yüksek @ 2016-05-19 22:45 UTC (permalink / raw
  To: gentoo-portage-dev

Repoman pulls in lxml unconditionally now and performs metadata checks
by default. This behavior makes these command line options obsolete
since forcing the default makes little sense. Declare them obsolete
instead of removing them for backwards compatibility.

Signed-off-by: Göktürk Yüksek <gokturk@gentoo.org>
---
 repoman/man/repoman.1            | 3 +--
 repoman/pym/repoman/argparser.py | 2 +-
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/repoman/man/repoman.1 b/repoman/man/repoman.1
index e2b9c66..fc5c52d 100644
--- a/repoman/man/repoman.1
+++ b/repoman/man/repoman.1
@@ -59,8 +59,7 @@ Be less verbose about extraneous info
 \fB-p\fR, \fB--pretend\fR
 Don't commit or fix anything; just show what would be done
 .TP
-\fB-x\fR, \fB--xmlparse\fR
-Forces the metadata.xml parse check to be carried out
+\fB-x\fR, \fB--xmlparse\fR (OBSOLETE)
 .TP
 \fB-v\fR, \fB--verbose\fR
 Displays every package name while checking
diff --git a/repoman/pym/repoman/argparser.py b/repoman/pym/repoman/argparser.py
index 2d56a87..3d35b6a 100644
--- a/repoman/pym/repoman/argparser.py
+++ b/repoman/pym/repoman/argparser.py
@@ -117,7 +117,7 @@ def parse_args(argv, qahelp, repoman_default_opts):
 	parser.add_argument(
 		'-x', '--xmlparse', dest='xml_parse', action='store_true',
 		default=False,
-		help='forces the metadata.xml parse check to be carried out')
+		help='(OBSOLETE)')
 
 	parser.add_argument(
 		'--if-modified', choices=('y', 'n'), default='n',
-- 
2.7.3



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

* [gentoo-portage-dev] Re: [PATCH] repoman: declare '-x', '--xmlparse' command line options obsolete
  2016-05-19 22:45 [gentoo-portage-dev] [PATCH] repoman: declare '-x', '--xmlparse' command line options obsolete Göktürk Yüksek
@ 2016-05-20  4:14 ` Duncan
  2016-05-20  9:10 ` [gentoo-portage-dev] " Alexander Berntsen
  1 sibling, 0 replies; 3+ messages in thread
From: Duncan @ 2016-05-20  4:14 UTC (permalink / raw
  To: gentoo-portage-dev

Göktürk Yüksek posted on Thu, 19 May 2016 18:45:29 -0400 as excerpted:

> Repoman pulls in lxml unconditionally now and performs metadata checks
> by default. This behavior makes these command line options obsolete
> since forcing the default makes little sense. Declare them obsolete
> instead of removing them for backwards compatibility.

I like the general idea, but not the implementation. =:^(

Example...

> -\fB-x\fR, \fB--xmlparse\fR
> -Forces the metadata.xml parse check to be carried out
> +\fB-x\fR, \fB--xmlparse\fR (OBSOLETE)


Often I'll find some online resource that recommends some command, but I 
(arguably wisely!) prefer to check the manpage to see what a command and 
its recommended options actually do, as opposed to just running it.  That 
way, in addition to protecting myself from rm -rf .* type advice as 
sometimes found online, I learn as I go and can then apply the new 
knowledge to similar situations, instead of being lost when an arbitrary 
command copied without understanding doesn't work.

Or maybe I'm simply trying to fix an old, poorly documented script that 
just broke, and am trying to figure out what some command therein 
actually does.

The problem is outdated options with no hint as to what they actually did 
before they were obsoleted and how to proceed with updating them for use 
with newer versions.  Were they obsoleted by more flexible options so the 
old version isn't needed but I need to figure out what new option to use 
and its format?  Is that behavior now the default?  Was that 
functionality removed and thus is no longer available?

So please, don't just declare it obsolete without saying what it actually 
did and if it applies, what the current equivalent might be.  Doing so is 
if anything even more frustrating to someone trying to figure out what 
the option actually did, than removing it from the documentation entirely!

So, perhaps (I'm not going to attempt formatting)...

--xmlparse (Obsolete, formerly forced a metadata.xml parse check but 
that's now default behavior.)

That way, anyone seeing the option somewhere in old code will still be 
able to lookup what it did, and know that they can simply mentally ignore 
that option when mentally tracing the old code in their head and delete 
it in their updated version, because it's now the default.

-- 
Duncan - List replies preferred.   No HTML msgs.
"Every nonfree program has a lord, a master --
and if you use the program, he is your master."  Richard Stallman



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

* Re: [gentoo-portage-dev] [PATCH] repoman: declare '-x', '--xmlparse' command line options obsolete
  2016-05-19 22:45 [gentoo-portage-dev] [PATCH] repoman: declare '-x', '--xmlparse' command line options obsolete Göktürk Yüksek
  2016-05-20  4:14 ` [gentoo-portage-dev] " Duncan
@ 2016-05-20  9:10 ` Alexander Berntsen
  1 sibling, 0 replies; 3+ messages in thread
From: Alexander Berntsen @ 2016-05-20  9:10 UTC (permalink / raw
  To: gentoo-portage-dev

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

Don't remove the description of what it did. Just declare it
deprecated.

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

iQIcBAEBCgAGBQJXPtSZAAoJENQqWdRUGk8B6QIP/A5MeyiDS4OPPieZwtVg7HcK
ifaoKoXL3okzAI9lejBev+7WSk19fo0BRyOgZUd1qzA6i2UHf9+RERMYGTxA+swX
qUOFwj+sF7piClaJp6QJy6R2lnlDgUn4V1q+UsNRxuOa+1vKUO+jRt8R9dsPDHtz
/Mfr94zesfLpg/R6l9ALqQHbTwCpakrS1o5LBzlwxXq+waVRaEAcmOmpecgosPcQ
AQsfKvGqdzG8m+m8TJkYsaNRsAVp5iml1L8EMpkMXzy7hbzWsWmi4uqtahpB7Lma
wxkfc4bfjVfJJHsJp1cg18klIqCJY9CP6AbYYB8w7eLkyVTQGIpZu303WJ5MSc+z
B2kMyCF4fLyG/UnKC5ae7jkUowXDjnmir17mVJQpqQ3W4w/oRJBhEzLRwp0YhN8G
BV7VcVFRs2t6FA6EGh+ieWNFl1YszijJmGWwV5t+t7FlBfArq4Srm/KdSfA+6WKP
VXLVKR41YUceiG2OZDWe4/sB6bDHjW0b6QFqgR6Ux58MmDP9llsK1vB4mMGBLteE
/SPs+IvpbLZQF5WuPZwARq8aLAcPHEbP4PwvH2kGqaOA0JPiHf7TOJGWEt/S41mr
iFFT95qZDDX5gPzlEBQi6sD6yPRiY3kEgjclCNE/1K9RyruwawLhFVnZJ/WXjabs
9Z3GF8dC0FgSiwu2jB+T
=taOP
-----END PGP SIGNATURE-----


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

end of thread, other threads:[~2016-05-20  9:10 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-05-19 22:45 [gentoo-portage-dev] [PATCH] repoman: declare '-x', '--xmlparse' command line options obsolete Göktürk Yüksek
2016-05-20  4:14 ` [gentoo-portage-dev] " Duncan
2016-05-20  9:10 ` [gentoo-portage-dev] " Alexander Berntsen

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