View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0004623GNUnetutil librarypublic2016-08-23 06:542019-07-24 18:42
ReporterlynX Assigned Toxrs Due Date
PrioritylowSeveritytweakReproducibilityalways
Status closedResolutionfixed 
Product VersionGit master 
Target Version0.11.6Fixed in Version0.11.6 
Summary0004623: UX: getopt should complain if -v is used with other parameters
DescriptionYou can require the user to understand why the version string is output,
or you can help them by telling them that the version string may only be
requested when there are no further parameters (which imply that the
user didn't actually intend to ask for a version string). Possibly this
kind of UX refinement is applicable to other parameters.

Also, is it normal that outputting a version string produces an error
return code? GNU tools usually do not.
Steps To Reproducegnunet-namestore -L DEBUG -z master-zone -ap -t vpn -e "2 hours" -n telnet -v "6 TVV54YYR29KRDKM30ZC3014JXHDGPVT7KFQGA44CEV2YQSEVFDAG telnet"
gnunet-namestore v0.10.1 svn-r37587
Makefile:142: recipe for target 'names' failed

The -v should have been -V to function, but a clearer error message would
have spared some time figuring out what was going on.
Additional InformationFork of 0004616
TagsNo tags attached.

Activities

lynX

2016-08-29 06:34

developer   ~0011072

The reason why I didn't just fix it is because when I looked over the code I realized it is all but trivial to make this change without a profound understanding of it. So if I was the right person to deal with this problem, I wouldn't have written this bug report.

Christian Grothoff

2016-08-29 08:16

manager   ~0011074

I suspect what's tripping you up here is 'getopt.c', which is actually GNU getopt from the copylib. I don't have any more profound understanding of the getopt.c file than you do, but that one should not be changed anyway, except for the last function in the file: that's the GNUnet-specific code. And of course you can change getopt_helpers.c.

dvn

2019-01-28 03:46

developer   ~0013488

This surely is a UX bug, at least. lynX any feedback to comment 0004623:0011074 ?

NOTE: I don't think this is a blocker for the 0.11.0 release, so I'm going to untag it.

Christian Grothoff

2019-01-28 04:59

manager   ~0013492

I disagree, it caused some serious confusion for lynx before, which is why he filed it. It also makes a nice exercise for beginners...

xrs

2019-06-22 21:32

reporter   ~0014569

Resolved by 988ac7f121b646cba3ca44567f54b1315398505d.

Issue History

Date ModifiedUsernameFieldChange
2016-08-23 06:54lynXNew Issue
2016-08-27 09:41Christian GrothoffAssigned To => lynX
2016-08-27 09:41Christian GrothoffStatusnew => assigned
2016-08-29 06:34lynXNote Added: 0011072
2016-08-29 06:34lynXAssigned TolynX => Christian Grothoff
2016-08-29 08:16Christian GrothoffNote Added: 0011074
2016-08-30 11:19Christian GrothoffAssigned ToChristian Grothoff =>
2016-08-30 11:19Christian GrothoffStatusassigned => confirmed
2016-09-22 18:10Christian GrothoffTarget Version => 0.11.0pre66
2017-02-21 17:38Christian GrothoffTarget Version0.11.0pre66 => 0.11.0
2017-02-26 01:26Christian GrothoffSeverityminor => feature
2018-06-26 22:02Christian GrothoffAssigned To => Feideus
2018-06-26 22:02Christian GrothoffStatusconfirmed => assigned
2019-01-27 10:59Christian GrothoffAssigned ToFeideus =>
2019-01-27 10:59Christian GrothoffStatusassigned => confirmed
2019-01-28 03:46dvnNote Added: 0013488
2019-01-28 03:47dvnSeverityfeature => tweak
2019-01-28 03:47dvnTarget Version0.11.0 =>
2019-01-28 04:59Christian GrothoffNote Added: 0013492
2019-06-05 17:02Christian GrothoffTarget Version => 0.11.6
2019-06-21 20:51Christian GrothoffAssigned To => t3sserakt
2019-06-21 20:51Christian GrothoffStatusconfirmed => assigned
2019-06-21 20:51Christian GrothoffAssigned Tot3sserakt => xrs
2019-06-22 21:32xrsStatusassigned => resolved
2019-06-22 21:32xrsResolutionopen => fixed
2019-06-22 21:32xrsFixed in Version => Git master
2019-06-22 21:32xrsNote Added: 0014569
2019-07-21 08:53Christian GrothoffFixed in VersionGit master => 0.11.6
2019-07-24 18:42Christian GrothoffStatusresolved => closed