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@xxxxxx 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review