Mbed-cli: Should not do full rebuild when switching profiles
Right now when you switch profiles it'll trigger a full rebuild. That is very annoying, I'm switching quite often between debug and release builds to look at memory usage etc. Can we not do this?
Downside is that changing the paths to BUILD/PLATFORM/GCC_ARM/PROFILE/ requires change in documentation.
cc @mark-edgeworth
janjongboom
All 11 comments
alternative might be to add '-profilename' to the end of either the platform or the compiler parts in the BUILD structure, although this feels a bit hacky. Another alternative is to maintain more than one BUILD structure, but logically the profile fits more with the device to me.
mark-edgeworth
on 18 Jan 2018
You could create a symbolic link for the old path every build. Would be a cheap way to keep the old paths working for a while. The symbolic links could be removed at a later timer (6.0?)
geky
on 19 Jan 2018
I don't know if Windows properly supports them.
janjongboom
on 19 Jan 2018
cc @theotherjimmy
screamerbg
on 29 Jan 2018
@janjongboom I don't think in re-builds assembly. I'll freely admit that is at best a 2% speedup. @geky is correct in pointing out that we would have to make the final artifact available as part of the BUILD/<target>/<toolchain>/ directory. Maybe just copying them would be sufficient? I'm open to ideas on this one.
theotherjimmy
on 29 Jan 2018
ARM Internal Ref: MBOTRIAGE-452
ciarmcom
on 6 Jun 2018
we would have to make the final artifact available as part of the
BUILD/<target>/<toolchain>/directory.
I think you would want to do something like
BUILD/<target>/<toolchain>/<build-profile-name-without-json-extension>/
as I intended to suggest on #700.
EDIT: Is this the same that @janjongboom was suggesting in the OP?
bmcdonnell-ionx
on 14 Jun 2018
You could create a symbolic link...
I don't know if Windows properly supports them.
Several (most?) versions of Windows require administrator rights to create symbolic links. I don't think you want to impose that requirement on your users. (Please don't.)
bmcdonnell-ionx
on 14 Jun 2018
@bmcdonnell-ionx Yes, exactly same proposal.
janjongboom
on 19 Jun 2018
712 has been merged to master, though not released yet.
Should this issue be closed?
bmcdonnell-ionx
on 6 Aug 2018
Yes.
theotherjimmy
on 24 Sep 2018
Related issues
maclobdell
路
3Comments
AlessandroA
路
6Comments
bmcdonnell-ionx
路
5Comments
stevew817
路
7Comments
JanneKiiskila
路
7Comments
Most helpful comment
Several (most?) versions of Windows require administrator rights to create symbolic links. I don't think you want to impose that requirement on your users. (Please don't.)