[Bug 230324] Review Request: avrdude -Software for programming Atmel AVR Microcontroller

bugzilla at redhat.com bugzilla at redhat.com
Wed Feb 28 12:34:49 UTC 2007


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: avrdude -Software for programming Atmel AVR Microcontroller


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230324





------- Additional Comments From rc040203 at freenet.de  2007-02-28 07:34 EST -------
(In reply to comment #1)
* The package builds against libusb if it finds it => Non deterministic builds

If you want to force building against libusb:
=> BuildRequires: libusb-devel

If you do NOT want to build against libusb
=> BuildConflicts: libusb-devel


* What also makes me wonder is the sources shipping a *.info but the package not
installing it ?!?


(In reply to comment #2)
> * Can you explain a bit about how the config file is not supposed to 
> be modified by end-users? 
I doubt Trond's statement. It's a system-wide configuration file, being
generated by the configure script, not a sample.
IMO, if it's a sample then it must not be located under /etc but should be
placed elsewhere (e.g. %doc)

For the moment I'd recommend to use
%configure ... --sysconfdir=%{_sysconfdir}/avrdude
And to mark it %config(noreplace)

i.e. to treat it as a system-wide config file.

Also not the source code in lexer.l: It refers to an avrdude.conf.sample which
doesn't exist (Minor upstream bug, I'd say).


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.




More information about the Fedora-package-review mailing list