Ticket #1842 (closed enhancement: fixed)

Opened 5 years ago

Last modified 3 years ago

Verbosity & informational improvements to dfu-util

Reported by: wiml Owned by: openmoko-devel
Priority: normal Milestone:
Component: host utilities Version:
Severity: normal Keywords: patch, HasPatch
Cc: Blocked By:
Blocking: Estimated Completion (week):
HasPatchForReview: yes PatchReviewResult:
Reproducible: always

Description

I was trying to figure out what dfu-util is doing wrong on my system. I made a handful of changes which I've included in this patch:

  • include underlying libusb error in dfu-util failure messages
  • make a second -v turn on verbose USB logging
  • document -v in the man page
  • extra details in error messages in a couple places
  • fix a couple of typos

Attachments

dfu-util-verbosity.patch​ (3.6 KB) - added by wiml 5 years ago.
make some of dfu-util's messages more informative

Change History

Changed 5 years ago by wiml

make some of dfu-util's messages more informative

comment:1 Changed 5 years ago by zecke

  • Keywords patch, HasPatch added; patch removed

comment:2 Changed 5 years ago by zecke

Could you please separate the spelling fixes and the verbosity patches into two patches?

comment:3 Changed 5 years ago by roh

  • HasPatchForReview set

BatchModify?: set HasPatchForReview? on 'keyword' contains 'patch'

comment:4 Changed 3 years ago by stefan

Hi wiml.

I'm late here, but I just took aver the maintainership of dfu-util.

There is a first release planned for dfu-util in mai and I would like to include your patches into it. Agreeing with Holger here it would be great if you could split up the patch and attach them here or send them to me via mail.

comment:5 Changed 3 years ago by stefan

  • Status changed from new to closed
  • Resolution set to fixed

I spliited your patch into three patches and applied them:

5957 - 5959, typo fix, debug output and man page update.

Thanks for your contribution.

Note: See TracTickets for help on using tickets.