Skip to content

Make build process easier to retarget to new Atmel release #43

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Dec 2, 2016

Conversation

sleemanj
Copy link
Contributor

@sleemanj sleemanj commented Dec 2, 2016

This pull request adds a file build.conf and various supporting changes to the build scripts.

The build.conf file allows you to specify simply the Atmel release version (AVR_VERSION) and that is the only change necessary to build for a new/different release from Atmel.

The version numbers of the supporting tools (autoconf...) and the gcc components not distributed by Atmel (gmp, mpfr, mpc) are automatically grepped out of the SOURCES.README in the Atmel distribution.

The URLs to these requirements are automatically constructed, and can be configured differently in the build.conf if you need to change that to a different mirror or whatever.

Also included is a slight fix to the build for automake which was failing on my system due to incompatibilities with texinfo - probably texinfo version specified by Atmel should be included into the tools build process, but maybe there is a catch-22 there.

…ct version numbers and so forth.

Just edit the first line of build.conf, then run...

  packag-avr-gcc.bash

or to just build...

  tools.bash
  binutils.build.bash
  gcc.build.bash
  avr-libc.build.bash
  gdb.build.bash

Add extra build docs.
@cmaglie
Copy link
Member

cmaglie commented Dec 2, 2016

Excellent, thank you!
Give me some time to test this PR, but I'm fully positive for merging.

@sleemanj
Copy link
Contributor Author

sleemanj commented Dec 2, 2016

NB: I've also added the one liner which solves #42

Copy link
Member

@cmaglie cmaglie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and working OK on my local PC (also the patch to gdb install).

@cmaglie
Copy link
Member

cmaglie commented Dec 2, 2016

Fixes #41 too

@facchinm facchinm merged commit f1a780d into arduino:master Dec 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants