[3437] in BarnOwl Developers
Re: [barnowl] Clean up Perl module build system (#129)
daemon@ATHENA.MIT.EDU (Anders Kaseorg)
Mon May 27 03:24:47 2013
Date: Mon, 27 May 2013 00:24:45 -0700
From: Anders Kaseorg <notifications@github.com>
Reply-To: barnowl/barnowl <reply+i-13015163-be40dbb946949d9a89d488109f91dfd2b6b71ced-4475081@reply.github.com>
To: barnowl/barnowl <barnowl@noreply.github.com>
In-Reply-To: <barnowl/barnowl/pull/129@github.com>
----==_mimepart_51a30a3d89707_62db5bdde439071
Date: Mon, 27 May 2013 00:24:45 -0700
Mime-Version: 1.0
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Content-ID: <51a30a3d8ba4b_62db5bdde4391e6@worker3.rs.github.com.mail>
So far the only comments I=E2=80=99ve gotten are of the form =E2=80=9Cyou=
should merge it and see what happens=E2=80=9D, so I plan to do that=E2=80=
=94but since this pull request predates the setup of pull request notific=
ations to barnowl-dev, as per the code review policy, I=E2=80=99m giving =
the list another chance to object or comment first.
Notable consequences of this branch:
* `Module::Install` is no longer bundled, and becomes a new build depende=
ncy. It=E2=80=99s packaged in all current versions of Debian and Ubuntu =
as `libmodule-install-perl`, and we can add it to the locker.
* Fixes to `Module::Install::BarnOwl` can be made in one place, not six.
* New BarnOwl modules can be added to the tree without copying a director=
y full of silly boilerplate files.
* We now pass `make distcheck` on Automake =E2=89=A5 1.7 (every version w=
e support), not just Automake =E2=89=A5 1.11.
---
Reply to this email directly or view it on GitHub:
https://github.com/barnowl/barnowl/pull/129#issuecomment-18485770=
----==_mimepart_51a30a3d89707_62db5bdde439071
Date: Mon, 27 May 2013 00:24:45 -0700
Mime-Version: 1.0
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Content-ID: <51a30a3d8c60e_62db5bdde439224@worker3.rs.github.com.mail>
<p>So far the only comments I=E2=80=99ve gotten are of the form =E2=80=9C=
you should merge it and see what happens=E2=80=9D, so I plan to do that=E2=
=80=94but since this pull request predates the setup of pull request noti=
fications to barnowl-dev, as per the code review policy, I=E2=80=99m givi=
ng the list another chance to object or comment first.</p>
<p>Notable consequences of this branch:</p>
<ul>
<li>
<code>Module::Install</code> is no longer bundled, and becomes a new buil=
d dependency. It=E2=80=99s packaged in all current versions of Debian an=
d Ubuntu as <code>libmodule-install-perl</code>, and we can add it to the=
locker.</li>
<li>Fixes to <code>Module::Install::BarnOwl</code> can be made in one pla=
ce, not six.</li>
<li>New BarnOwl modules can be added to the tree without copying a direct=
ory full of silly boilerplate files.</li>
<li>We now pass <code>make distcheck</code> on Automake =E2=89=A5 1.7 (ev=
ery version we support), not just Automake =E2=89=A5 1.11.</li>
</ul>
<p style=3D"font-size:small;-webkit-text-size-adjust:none;color:#666;">&m=
dash;<br>Reply to this email directly or <a href=3D'https://github.com/ba=
rnowl/barnowl/pull/129#issuecomment-18485770'>view it on GitHub</a>.<img =
src=3D'https://github.com/notifications/beacon/JJk3yKd0u6qAmPAmJXdf90MQrA=
DAfK1mD6IUy06iagE_RT_O9TEdGV7rFarg3iOp.gif' height=3D'1' width=3D'1'></p>=
----==_mimepart_51a30a3d89707_62db5bdde439071--
