From: Junio C Hamano <gitster@pobox.com>
To: "Ævar Arnfjörð Bjarmason" <avarab@gmail.com>
Cc: git@vger.kernel.org, Linux Kernel <linux-kernel@vger.kernel.org>,
git-packagers@googlegroups.com
Subject: Re: [ANNOUNCE] Git v2.17.0-rc0
Date: Fri, 16 Mar 2018 11:05:45 -0700 [thread overview]
Message-ID: <xmqqbmfn3ngm.fsf@gitster-ct.c.googlers.com> (raw)
In-Reply-To: <87fu50e0ht.fsf@evledraar.gmail.com> ("Ævar Arnfjörð Bjarmason"'s message of "Fri, 16 Mar 2018 12:14:06 +0100")
Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:
> On Fri, Mar 16 2018, Junio C. Hamano jotted:
>
>> gitweb: hard-depend on the Digest::MD5 5.8 module
>
> I've just noticed this now, but while this module is in 5.8 RedHat's
> butchered perl doesn't have it in the base system, thus this introduces
> the do-we-even-care regression that git's full test suite won't pass on
> a RedHat (or CentOS) base system, because the gitweb tests will fail to
> "use" Digest::MD5.
>
> I'm slightly leaning towards not caring about it, since there's no other
> perl distributor that does this sort of split-out of the core, and if
> you're on a RedHat system they're solving your package problems, so this
> really only impacts the edge case of git developers and redhat
> packagers, both of whom can just do "yum install -y perl-Digest-MD5" to
> fix it.
Thanks for noting. I agree that this is not something that requires
more than a mention near the beginning of release notes.
I haven't wordsmithed it fully, but it should say something along
the lines of ...
Documentation/RelNotes/2.16.0.txt | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/Documentation/RelNotes/2.16.0.txt b/Documentation/RelNotes/2.16.0.txt
index 8f0461eefd..8b4c24200b 100644
--- a/Documentation/RelNotes/2.16.0.txt
+++ b/Documentation/RelNotes/2.16.0.txt
@@ -6,6 +6,16 @@ Backward compatibility notes and other notable changes.
* Use of an empty string as a pathspec element that is used for
'everything matches' is now an error.
+ * Part of Git that depends on Perl have required at least Perl 5.8
+ since Git v1.7.4 released in 2010, but we used to assume some core
+ modules from Perl distribution may not exist on the system and did
+ a conditional "eval { require <<module>> }"; we no longer do this.
+ On a platform that ships a stripped-down Perl by default, the user
+ may have to install modules the platform chooses not to ship as
+ part of its core (e.g. Digest::MD5, File::Temp, File::Spec,
+ Net::SMTP, NET::Domain). RedHat/CentOS excludes Digest::MD5 from
+ its base installation, for example.
+
Updates since v2.15
-------------------
next prev parent reply other threads:[~2018-03-16 18:06 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-16 0:57 [ANNOUNCE] Git v2.17.0-rc0 Junio C Hamano
2018-03-16 11:14 ` Ævar Arnfjörð Bjarmason
2018-03-16 18:05 ` Junio C Hamano [this message]
2018-03-16 18:08 ` Junio C Hamano
2018-03-16 22:07 ` [PATCH] RelNotes: add details on Perl module changes Todd Zullinger
2018-03-16 22:12 ` Todd Zullinger
2018-03-16 22:19 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=xmqqbmfn3ngm.fsf@gitster-ct.c.googlers.com \
--to=gitster@pobox.com \
--cc=avarab@gmail.com \
--cc=git-packagers@googlegroups.com \
--cc=git@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).