Merge lp://qastaging/~fboudra/linaro-ci/various-fixes-2 into lp://qastaging/linaro-ci

Proposed by Fathi Boudra
Status: Merged
Approved by: Milo Casagrande
Approved revision: 137
Merged at revision: 136
Proposed branch: lp://qastaging/~fboudra/linaro-ci/various-fixes-2
Merge into: lp://qastaging/linaro-ci
Diff against target: 75 lines (+14/-11)
2 files modified
jenkins_common_lib (+4/-3)
jenkins_kernel_build_inst (+10/-8)
To merge this branch: bzr merge lp://qastaging/~fboudra/linaro-ci/various-fixes-2
Reviewer Review Type Date Requested Status
Milo Casagrande (community) Approve
Review via email: mp+155176@code.qastaging.launchpad.net

Description of the change

Various fixes:
- replace git describe by make version to get kernel version (LP: #1092037)
- get rid of GIT_COMMIT (provided by jenkins git plugins) in favor of git log
- avoid to duplicate info in GIT_LOG (the commit id is already provided through KERNEL_GIT)

To post a comment you must log in.
Revision history for this message
Milo Casagrande (milo) wrote :

As discussed on IRC with Fathi, it is good to go with a small revert.
+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches