On Fri, Jul 12, 2013 at 09:48:12AM -0400, David Nalley wrote: > On Thu, Jul 11, 2013 at 11:05 PM, Alex Huang <alex.hu...@citrix.com> wrote: > > Will work on making it nicer but do we want to put this information into > > release? Is there any harm to have this in the release build? I can't > > think of any off-hand. If we don't want them, we can change build_asf.sh > > to set those two variables to something nicer. If we want them, I might > > be able to get it to read this information from a file instead. > > > > --Alex > > > > Thanks for putting the work in to get this done. > I agree - no harm in having this in the release build, and could be > very useful to boot. > I personally would like to see this information, present - setting it > at tarball creation makes sense to me. > > --David >
+1 The only reason it was removed previously was that it was breaking the build process when someone tried to build from the release tarball. If we want to drop the details into the release archive before tar'ing, great! The only other minor concern would be the release verification process will have to be updated to account for the fact that we'll have one file (or some lines) that are different between the actual tagged commit in the repo and what's in the tarball. As far as I know, that's just a process change for us. Easy... -chip