[Bug 238705] Review Request: avr-gcc - Cross Compiling GNU GCC targeted at avr

bugzilla at redhat.com bugzilla at redhat.com
Wed May 2 14:41:10 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: avr-gcc - Cross Compiling GNU GCC targeted at avr


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





------- Additional Comments From j.w.r.degoede at hhs.nl  2007-05-02 10:41 EST -------
(In reply to comment #1)
> Several questions based on visual inspection of the spec:
> 
> * Why are you using "make all-gcc" and "make install-gcc" instead of "make all"
> and "make install"?
> 

Good question, I don't know my students wrote the initial version and used this.
Unless they can give me a good reason I'll revert to the normal make targets.

> * Did you consider to split gcc and gcc-c++?
> Mixing c and c++ into one package when many users of embedded system will only
> use c implies a pretty ugly package bloat on them.
> 

I didn't think about this, a very good point I'll put them in a sub package.

> * If I am informed correctly (I am only semi-educated on the avr toolchain), the
> classical avr toolchain is based on avr-libc. This would imply to build GCC
> against avr libc instead of "no libc".

AFAIK the 2-stage building of gcc is not necessary for the AVR, none of the docs
on compiling gcc for the AVR I could find talk about this. See for example:
http://www.nongnu.org/avr-libc/user-manual/install_tools.html

Re-reading this page, I do notice that I forgot to add these 2 adviced configure
flags: "--disable-libssp --with-dwarf2". Unless someone objects I'll add these,
although I think --with-dwarf2 is only needed under windows for AVRStudio
compatibility though, so I'm not sure if thats needed for us.


-- 
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