Skip to content
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

[server] write paparzzi_version as comment at beginning of log #1049

Merged
merged 1 commit into from
Dec 19, 2014

Conversation

flixr
Copy link
Member

@flixr flixr commented Dec 18, 2014

Writes an xml comment at the beginning of the *.log file, e.g.
<!-- logged with paparazzi_version v5.3.2_testing-1-gbdf9a8f -->

@flixr
Copy link
Member Author

flixr commented Dec 18, 2014

Please note that this will add the paparazzi_version at run time and not at build time of the server...

@dewagter
Copy link
Member

here also there is a difference between the version when you last did
"make" and the version when you run the thing. Especially when switching
branches and forgetting to make this has caused me issues before. So it
might be nice to add both while we are at it...

-Christophe

On Fri, Dec 19, 2014 at 12:43 AM, Felix Ruess [email protected]
wrote:

Please note that this will add the paparazzi_version at run time and not
at build time of the server...


Reply to this email directly or view it on GitHub
#1049 (comment).

@flixr
Copy link
Member Author

flixr commented Dec 19, 2014

I have to leave it to @gautierhattenberger to add the build time version in OCaml..

@flixr flixr merged commit ec82874 into master Dec 19, 2014
@flixr flixr added this to the v5.4 milestone Dec 19, 2014
@flixr flixr deleted the log_version branch December 19, 2014 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants