<feed xmlns='http://www.w3.org/2005/Atom'>
<title>git/Makefile, branch v2.17.2</title>
<subtitle>Mirror of https://git.kernel.org/pub/scm/git/git.git/
</subtitle>
<id>https://git.shady.money/git/atom?h=v2.17.2</id>
<link rel='self' href='https://git.shady.money/git/atom?h=v2.17.2'/>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/'/>
<updated>2018-03-15T22:00:46Z</updated>
<entry>
<title>Merge branch 'ab/perl-fixes'</title>
<updated>2018-03-15T22:00:46Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2018-03-15T22:00:46Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=ae1644b08e85fb64367b273337d8ebcbdabe0685'/>
<id>urn:sha1:ae1644b08e85fb64367b273337d8ebcbdabe0685</id>
<content type='text'>
Clean-up to various pieces of Perl code we have.

* ab/perl-fixes:
  perl Git::LoadCPAN: emit better errors under NO_PERL_CPAN_FALLBACKS
  Makefile: add NO_PERL_CPAN_FALLBACKS knob
  perl: move the perl/Git/FromCPAN tree to perl/FromCPAN
  perl: generalize the Git::LoadCPAN facility
  perl: move CPAN loader wrappers to another namespace
  perl: update our copy of Mail::Address
  perl: update our ancient copy of Error.pm
  git-send-email: unconditionally use Net::{SMTP,Domain}
  Git.pm: hard-depend on the File::{Temp,Spec} modules
  gitweb: hard-depend on the Digest::MD5 5.8 module
  Git.pm: add the "use warnings" pragma
  Git.pm: remove redundant "use strict" from sub-package
  perl: *.pm files should not have the executable bit
</content>
</entry>
<entry>
<title>Merge branch 'ab/simplify-perl-makefile'</title>
<updated>2018-03-06T22:54:04Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2018-03-06T22:54:04Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=f65570719458833c20dfdee760bceee069c09056'/>
<id>urn:sha1:f65570719458833c20dfdee760bceee069c09056</id>
<content type='text'>
Hotfix for a topic already in 'master'.

* ab/simplify-perl-makefile:
  Makefile: generate Git(3pm) as dependency of the 'doc' and 'man' targets
</content>
</entry>
<entry>
<title>perl Git::LoadCPAN: emit better errors under NO_PERL_CPAN_FALLBACKS</title>
<updated>2018-03-05T18:52:28Z</updated>
<author>
<name>Ævar Arnfjörð Bjarmason</name>
<email>avarab@gmail.com</email>
</author>
<published>2018-03-03T15:38:17Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=1aca69c0195bd4ac671e7a36cf9b967d2bcbe200'/>
<id>urn:sha1:1aca69c0195bd4ac671e7a36cf9b967d2bcbe200</id>
<content type='text'>
Before my 20d2a30f8f ("Makefile: replace perl/Makefile.PL with simple
make rules", 2017-12-10) on an OS package that removed the
private-Error.pm copy we carried around manually removing the OS's
Error.pm would yield:

    $ git add -p
    Can't locate Error.pm in @INC (you may need to install the Error module) [...]

Now, before this change we'll instead emit this more cryptic error:

    $ git add -p
    BUG: '/usr/share/perl5/Git/FromCPAN' should be a directory! at /usr/share/perl5/Git/Error.pm line 36.

This is a confusing error. Now if the new NO_PERL_CPAN_FALLBACKS
option is specified and we can't find the module we'll instead emit:

    $ /tmp/git/bin/git add -p
    BUG: The 'Error' module is not here, but NO_PERL_CPAN_FALLBACKS was set!

    [...]

Where [...] is the lengthy explanation seen in the change below, which
explains what the potential breakage is, and how to fix it.

The reason for checking @@NO_PERL_CPAN_FALLBACKS@@] against the empty
string in Perl is as opposed to checking for a boolean value is that
that's (as far as I can tell) make's idea of a string that's set, and
e.g. NO_PERL_CPAN_FALLBACKS=0 is enough to set NO_PERL_CPAN_FALLBACKS.

Signed-off-by: Ævar Arnfjörð Bjarmason &lt;avarab@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Makefile: add NO_PERL_CPAN_FALLBACKS knob</title>
<updated>2018-03-05T18:52:28Z</updated>
<author>
<name>Todd Zullinger</name>
<email>tmz@pobox.com</email>
</author>
<published>2018-03-03T15:38:16Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=075321c0075b1ef86b98facc88bcaeefeb9584d1'/>
<id>urn:sha1:075321c0075b1ef86b98facc88bcaeefeb9584d1</id>
<content type='text'>
We include some perl modules which are not part of the core perl
install, as a convenience.  This allows us to rely on those modules in
our perl-based tools and scripts without requiring users to install the
modules from CPAN or their operating system packages.

Users whose operating system provides these modules and packagers of Git
often don't want to ship or use these bundled modules.  Allow these
users to set NO_PERL_CPAN_FALLBACKS to avoid installing the bundled
modules.

Signed-off-by: Todd Zullinger &lt;tmz@pobox.com&gt;
Signed-off-by: Ævar Arnfjörð Bjarmason &lt;avarab@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>perl: move the perl/Git/FromCPAN tree to perl/FromCPAN</title>
<updated>2018-03-05T18:52:28Z</updated>
<author>
<name>Ævar Arnfjörð Bjarmason</name>
<email>avarab@gmail.com</email>
</author>
<published>2018-03-03T15:38:15Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=382029fc003d400147339573ab5448a88c336f55'/>
<id>urn:sha1:382029fc003d400147339573ab5448a88c336f55</id>
<content type='text'>
Move the CPAN modules that have lived under perl/Git/FromCPAN since my
20d2a30f8f ("Makefile: replace perl/Makefile.PL with simple make
rules", 2017-12-10) to perl/FromCPAN.

A subsequent change will teach the Makefile to only install these
copies of CPAN modules if a flag that distro packagers would like to
set isn't set. Due to how the wildcard globbing is being done it's
much easier to accomplish that if they're moved to their own
directory.

Signed-off-by: Ævar Arnfjörð Bjarmason &lt;avarab@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'tz/do-not-clean-spec-file'</title>
<updated>2018-02-28T21:37:58Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2018-02-28T21:37:58Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=09967278795fa0b36e7f1308c2063005a263d936'/>
<id>urn:sha1:09967278795fa0b36e7f1308c2063005a263d936</id>
<content type='text'>
We no longer create any *.spec file, so "make clean" should not
remove it.

* tz/do-not-clean-spec-file:
  Makefile: remove *.spec from clean target
</content>
</entry>
<entry>
<title>Merge branch 'rj/sparse-updates'</title>
<updated>2018-02-27T18:33:55Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2018-02-27T18:33:55Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=14599b48c0a145452db4363f4eee4d3fc6187d05'/>
<id>urn:sha1:14599b48c0a145452db4363f4eee4d3fc6187d05</id>
<content type='text'>
Devtool update.

* rj/sparse-updates:
  Makefile: suppress a sparse warning for pack-revindex.c
  config.mak.uname: remove SPARSE_FLAGS setting for cygwin
</content>
</entry>
<entry>
<title>Makefile: remove *.spec from clean target</title>
<updated>2018-02-17T21:27:57Z</updated>
<author>
<name>Todd Zullinger</name>
<email>tmz@pobox.com</email>
</author>
<published>2018-02-17T03:06:24Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=4321bdcabb481ba49eb79f332881d19950f72d4b'/>
<id>urn:sha1:4321bdcabb481ba49eb79f332881d19950f72d4b</id>
<content type='text'>
Support for generating an rpm was dropped in ab214331cf ("Makefile: stop
pretending to support rpmbuild", 2016-04-04).  We don't generate any
*.spec files so there is no need to clean them up.

Signed-off-by: Todd Zullinger &lt;tmz@pobox.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'ab/sha1dc-build'</title>
<updated>2018-02-15T22:55:40Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2018-02-15T22:55:40Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=a66b51c624d9d8700b57646322d23231d01b9be8'/>
<id>urn:sha1:a66b51c624d9d8700b57646322d23231d01b9be8</id>
<content type='text'>
Push the submodule version of collision-detecting SHA-1 hash
implementation a bit harder on builders.

* ab/sha1dc-build:
  sha1dc_git.h: re-arrange an ifdef chain for a subsequent change
  Makefile: under "make dist", include the sha1collisiondetection submodule
  Makefile: don't error out under DC_SHA1_EXTERNAL if DC_SHA1_SUBMODULE=auto
</content>
</entry>
<entry>
<title>Makefile: generate Git(3pm) as dependency of the 'doc' and 'man' targets</title>
<updated>2018-02-15T18:05:32Z</updated>
<author>
<name>SZEDER Gábor</name>
<email>szeder.dev@gmail.com</email>
</author>
<published>2018-02-15T02:14:10Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=2530afd3519a34b66e72cc29e7751d650cedc6dd'/>
<id>urn:sha1:2530afd3519a34b66e72cc29e7751d650cedc6dd</id>
<content type='text'>
Since commit 20d2a30f8f (Makefile: replace perl/Makefile.PL with
simple make rules, 2017-12-10), the Git(3pm) man page is only
generated as an indirect dependency of the 'install-doc' and
'install-man' Makefile targets.  Consequently, if someone runs 'make
man &amp;&amp; sudo make install-man' (or their 'doc' counterparts), then
Git(3pm) will be generated as root, and the resulting root-owned files
and directories will in turn cause the next user-run 'make clean' to
fail.  This was not an issue in the past, because Git(3pm) was
generated when 'make all' descended into 'perl/', which is usually not
run as root.

List Git(3pm) as a dependency of the 'doc' and 'man' Makefile targets,
too, so it gets generated by targets that are usually built as
ordinary users.

While at it, add 'install-man-perl' to the list of .PHONY targets.

Signed-off-by: SZEDER Gábor &lt;szeder.dev@gmail.com&gt;
Acked-by: Ævar Arnfjörð Bjarmason &lt;avarab@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
</feed>
