forked from arduino/Arduino
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix build to properly copy WiFi examples for EMT CC3200 WiFi library
- Loading branch information
Showing
1 changed file
with
5 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
8c25e7c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess you guys must never do:
If you do, then the target directory doesn't exist ${target.path}/hardware/emt/ti/runtime/wiring/cc3200/libraries/WiFi and the build fails.
I have to do this now:
8c25e7c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hummm... I'll take a look.
FWIW: on every repo update we do an
ant <os>-clean; ant dist
where
<os>
islinux
,windows
, ...8c25e7c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
doesn't seem to clean the work directory. There is lots of stuff left in there after the clean
That basically deletes the work directory
8c25e7c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Our build logs show the same ant output and, in a quick review of
build.xml
, it appears thatlinux-clean
andclean
are the same when running on linux. Thinking it must be the difference betweenbuild
anddist
targets, I replacedlinux-clean
withclean
,dist
withbuild
, and rebuilt on linux. I still couldn't reproduce the failure.So, for now ... you should be able to workaround the issue by adding the following line to
build.xml
just prior to the offending line
Please give this a try and let me know if it helps. Also, any additional details plus the ant error messages would be helpful.
8c25e7c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had modified my build.xml with that exact same fix prior to writing the comment when this problem first appeared with the Feb 15th checkin , and yes that fixes it.
However, I was more concerned that your builds aren't failing. I don't know what source or branch you are working with but doing a clean checkout and build doesn't work for me. I followed the instructions from https://github.com/energia/Energia/blob/master/ReadMe.md
BTW: git pull fails in the emt directory
8c25e7c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, me too.
Thanks for the build log, I think I now know what's happening: We're building with a later version of the closure (
closure-emt-b11.tar.gz
rather thanclosure-emt-a61.tar.gz
) which is targeted for the next release of Energia but have not yet updated the.tar.gz
that's in the Energia repo.The Energia repo is huge and takes forever to clone. So, our automated builds/regressions don't clone from scratch and I've been avoiding adding more binary blobs that only makes the problem worse.
8c25e7c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
8c25e7c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From here? http://energia.nu/files/closure-emt-b11.tar.gz
If so, them you must be using a different build.xml .. unless you have renamed b11 to a61 ...
So if I modify build.xml and change emt-a61 to emt-b11 it builds properly now. I didn't try any of the multi tasking examples. However, the crux of the problem is the checked in source code being out of sync with what you are actually using to build and test. The b11 archive has the missing directory, the a61 archive does not.
8c25e7c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. Every time there's a change to emt, this tar ball is automatically updated (and the version number is incremented).
Our builds download the latest version and pass it on the command line:
ant -Dclosure-version=emt-b11 dist
Thanks for verifying this (and pointing out the problem). I'll fix
build.xml
to add the missing directory so others don't run into the same issue and you can use a clean clone.8c25e7c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you see my comment about how to speed up your clones by limiting the depth? On my really old machine it takes less than 2 minutes to do a clone
8c25e7c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That would have been useful info ... Thanks that does work
8c25e7c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.