Improve WTMI build for git-less systems.#30
Conversation
This is particularly useful when the sources are packaged in a tarball. Signed-off-by: Matwey V. Kornilov <matwey.kornilov@gmail.com>
pali
left a comment
There was a problem hiding this comment.
The point here is to include full version of WTMI into BUILD_STRING which is printed by wtmi firmware at runtime over UART. There are too many WTMI builds on internet and this version string, including commit id, is the only identifier to figure out what the is the exact version of running WTMI version. When debugging WTMI firmware, people just put debug log and so it is required to have this string in it. And I helped to debug more issues in WTMI just based on version strings.
This change is removing version and commit id and therefore makes debugging of community builds harder or probably impossible. So it does not improve it but instead makes it worse.
Note that there are no tarballs of WTMI (anymore) so this tarball argument does not have any sense. In case you are using Marvell SDK tarballs (which are under NDA) then ask your Marvell representative and not community builds.
Due to debugging issues with community builds I'm against this change as it basically disallow me to debug issues in community builds of this firmware.
This is particularly useful when the sources are packaged in a tarball.
Signed-off-by: Matwey V. Kornilov matwey.kornilov@gmail.com