Re: [PATCH] Build process makeover

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2009-10-01 13:42, FUJITA Tomonori wrote:
> Thanks, the changes look fine but seems that it doesn't work as I
> expected.
> 
> fujita@viola:/tmp/tgt$ git-am ~/Mail/inbox/235
> Applying: Build process makeover
> /tmp/tgt/.git/rebase-apply/patch:23: trailing whitespace.
> # line).
> /tmp/tgt/.git/rebase-apply/patch:49: trailing whitespace.
> programs:
> /tmp/tgt/.git/rebase-apply/patch:52: trailing whitespace.
> install-programs:
> /tmp/tgt/.git/rebase-apply/patch:60: trailing whitespace.
> doc:
> /tmp/tgt/.git/rebase-apply/patch:63: trailing whitespace.
> install-doc:
> warning: squelched 10 whitespace errors
> warning: 15 lines add whitespace errors.
> 
> I can fix whitespace warnings (but please fix them next time)

Sorry. Will do.

> 
> fujita@viola:/tmp/tgt$ git-tag 0.9.10
> fujita@viola:/tmp/tgt$ make tarball
> 
> fujita@viola:/tmp/tgt$ LANG=C make tarball
> Makefile:13: version-stamp: No such file or directory
> echo "VERSION ?= `git describe --tags HEAD | sed -e 's/^v//' -e 's/-/_/g'`" > version-stamp
> make -j4 -C usr clean
> make[1]: Entering directory `/tmp/tgt/usr'
> rm -f *.[od] tgtd tgtadm tgtimg iscsi/*.[od] ibmvio/*.[od] fc/*.[od] fcoe/*.[od]
> make[1]: Leaving directory `/tmp/tgt/usr'
> make -j4 -C doc clean
> make[1]: Entering directory `/tmp/tgt/doc'
> make[1]: Nothing to be done for `clean'.
> make[1]: Leaving directory `/tmp/tgt/doc'
> make -j4 -C conf clean
> make[1]: Entering directory `/tmp/tgt/conf'
> make[1]: Nothing to be done for `clean'.
> make[1]: Leaving directory `/tmp/tgt/conf'
> make -j4 -C scripts clean
> make[1]: Entering directory `/tmp/tgt/scripts'
> make[1]: Nothing to be done for `clean'.
> make[1]: Leaving directory `/tmp/tgt/scripts'
> rm -f tgt-0.9.10.tar.bz2
> make -j4 -C usr
> make[1]: Entering directory `/tmp/tgt/usr'
> cc -c -DUSE_SIGNALFD -D_GNU_SOURCE -I. -g -O2 -Wall -Wstrict-prototypes -fPIC -DTGT_VERSION=\"0.9.10\" tgtd.c -o tgtd.o
> cc -c -DUSE_SIGNALFD -D_GNU_SOURCE -I. -g -O2 -Wall -Wstrict-prototypes -fPIC -DTGT_VERSION=\"0.9.10\" mgmt.c -o mgmt.o
> cc -c -DUSE_SIGNALFD -D_GNU_SOURCE -I. -g -O2 -Wall -Wstrict-prototypes -fPIC -DTGT_VERSION=\"0.9.10\" target.c -o target.o
> 
> 
> Hmm, why do I need to compile to build a tarball?

Urgh. tarball definitely should not depend on "all". Must have slipped
in at the last moment. Sorry. Fix on the way.

> Then,
> 
> fujita@viola:/tmp/tgt$ tar tjf tgt-0.9.10.tar.bz2
> tgt-0.9.10/Makefile
> tgt-0.9.10/README
> ...
> tgt-0.9.10/usr/libcrc32c.o
> ...
> 
> The tarball should not include object files.

Yes, that is of course nonsense. Same reason as above.

> Can't we avoid putting version-stamp file in tarballs (e.g. Can we set
> the proper VERSION in Makefile)?

I hate the idea of sed'ing around in a Makefile. The .include
version-stamp magic was the best thing I could think of. If you have a
better suggestion, then please do share.

> And why can't we use git-archive?

Doesn't git archive work only for files under git control? If so, then
that would work only if we come with an alternative to version-stamp.
Plus, the rpm branch autogenerates a spec file from a template, and then
puts that spec file inside the tarball, so that wouldn't work with
git-archive either. And _not_ including the spec file breaks support for
"rpmbuild -tb".

Cheers,
Florian

Attachment: signature.asc
Description: OpenPGP digital signature


[Index of Archives]     [Linux SCSI]     [Linux RAID]     [Linux Clusters]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]

  Powered by Linux