arduino / toolchain-avr

The AVR toolchain used by the Arduino IDE
144 stars 48 forks source link

package-avr-gcc.bash: Please, inform me before to delete all my work : -) #78

Open ghost opened 3 years ago

ghost commented 3 years ago

Hello,

After a tedious but successful execution of tools.bash, binutils.build.bash, gcc.build.bash, avr-libc.build.bash and gdb.build.bash it is time for me to package the libs and programs generated in objdir. According to the doc the script for that is package-avr-gcc.bash.

A) Arrggg! package-avr-gcc.bash starts by deleting all that has been done before, including objdir. Then it rebuilds everything from scratch and finally makes a package.

Please, at least, update the doc to explain that!

May I also suggest this enhancement:

  1. Duplicate package-avr-gcc.bash in a new file: all.build.bash or full.build.bash (or any other name making obvious that the script does all that is required to generate the toolchain).
  2. Remove from package-avr-gcc.bash everything that is not related to packaging (this makes the name of the script consistent with its action).
  3. In all.build.bash replace everything related to packaging by a call to package-avr-gcc.bash.
  4. Last but not least, update the doc to explain there are two ways to build the toolchain: a. The step by step method: tools.bash, binutils.build.bash, gcc.build.bash, avr-libc.build.bash, gdb.build.bash and package-avr-gcc.bash. b. The single step method using only all.build.bash.

B) Under Cygwin the script ./package-avr-gcc.bash generates errors when it rebuilds everything from scratch, even if each step was working fine individually. I will create other issues for these errors.

Regards.