Re: [HACKERS] Small PATCH: check of 2 Perl modules

2016-03-01 Thread Robert Haas
On Tue, Feb 16, 2016 at 11:48 AM, Victor Wagner wrote: > On Tue, 16 Feb 2016 12:23:56 -0300 > Alvaro Herrera wrote: > >> ... but I agree with the point upthread that this should wait to see >> what happens with the CMake stuff, since this is not a newly >> introduced problem. > > I doubt, that CM

Re: [HACKERS] Small PATCH: check of 2 Perl modules

2016-02-16 Thread Victor Wagner
On Tue, 16 Feb 2016 12:23:56 -0300 Alvaro Herrera wrote: > ... but I agree with the point upthread that this should wait to see > what happens with the CMake stuff, since this is not a newly > introduced problem. I doubt, that CMake stuff would be ready for 9.6. There is just one commitfest left

Re: [HACKERS] Small PATCH: check of 2 Perl modules

2016-02-16 Thread Alvaro Herrera
Victor Wagner wrote: > Not everyone have "standard perl installation" nowadays. Most Linux > users, for example, use Perl package from the distrubution, and > distributions love to strip down standard perl installation putting its > parts into separate packages, some of which might be optional. >

Re: [HACKERS] Small PATCH: check of 2 Perl modules

2016-02-16 Thread Victor Wagner
On Mon, 15 Feb 2016 08:34:11 -0500 Peter Eisentraut wrote: > On 2/12/16 8:20 AM, Eugene Kazakov wrote: > > TAP-tests need two Perl modules: Test::More and IPC::Run. > > > > The patch adds checking of modules in configure.in and configure. > > I think those modules are part of a standard Perl

Re: [HACKERS] Small PATCH: check of 2 Perl modules

2016-02-15 Thread Peter Eisentraut
On 2/15/16 8:57 AM, Teodor Sigaev wrote: >> On 2/12/16 8:20 AM, Eugene Kazakov wrote: >>> TAP-tests need two Perl modules: Test::More and IPC::Run. >> I think those modules are part of a standard Perl installation. > > IPC::Run is not. At least for perl from ports tree in FreeBSD. Right, that's w

Re: [HACKERS] Small PATCH: check of 2 Perl modules

2016-02-15 Thread Tom Lane
Teodor Sigaev writes: >> On 2/12/16 8:20 AM, Eugene Kazakov wrote: >>> TAP-tests need two Perl modules: Test::More and IPC::Run. >> I think those modules are part of a standard Perl installation. > IPC::Run is not. At least for perl from ports tree in FreeBSD. Yeah, I remember having had to ins

Re: [HACKERS] Small PATCH: check of 2 Perl modules

2016-02-15 Thread Michael Paquier
On Mon, Feb 15, 2016 at 10:57 PM, Teodor Sigaev wrote: >> On 2/12/16 8:20 AM, Eugene Kazakov wrote: >>> >>> TAP-tests need two Perl modules: Test::More and IPC::Run. >> >> I think those modules are part of a standard Perl installation. > > IPC::Run is not. At least for perl from ports tree in Free

Re: [HACKERS] Small PATCH: check of 2 Perl modules

2016-02-15 Thread Teodor Sigaev
On 2/12/16 8:20 AM, Eugene Kazakov wrote: TAP-tests need two Perl modules: Test::More and IPC::Run. I think those modules are part of a standard Perl installation. IPC::Run is not. At least for perl from ports tree in FreeBSD. -- Teodor Sigaev E-mail: teo...

Re: [HACKERS] Small PATCH: check of 2 Perl modules

2016-02-15 Thread Peter Eisentraut
On 2/12/16 8:20 AM, Eugene Kazakov wrote: > TAP-tests need two Perl modules: Test::More and IPC::Run. > > The patch adds checking of modules in configure.in and configure. I think those modules are part of a standard Perl installation. -- Sent via pgsql-hackers mailing list (pgsql-hackers@post

Re: [HACKERS] Small PATCH: check of 2 Perl modules

2016-02-15 Thread Eugene Kazakov
13.02.2016 16:04, Michael Paquier : On Sat, Feb 13, 2016 at 1:47 PM, Robert Haas wrote: On Fri, Feb 12, 2016 at 8:20 AM, Eugene Kazakov wrote: TAP-tests need two Perl modules: Test::More and IPC::Run. The patch adds checking of modules in configure.in and configure. Why would we want that?

Re: [HACKERS] Small PATCH: check of 2 Perl modules

2016-02-13 Thread Michael Paquier
On Sat, Feb 13, 2016 at 1:47 PM, Robert Haas wrote: > On Fri, Feb 12, 2016 at 8:20 AM, Eugene Kazakov > wrote: >> TAP-tests need two Perl modules: Test::More and IPC::Run. >> >> The patch adds checking of modules in configure.in and configure. > > Why would we want that? I was doubtful at the be

Re: [HACKERS] Small PATCH: check of 2 Perl modules

2016-02-12 Thread Robert Haas
On Fri, Feb 12, 2016 at 8:20 AM, Eugene Kazakov wrote: > TAP-tests need two Perl modules: Test::More and IPC::Run. > > The patch adds checking of modules in configure.in and configure. Why would we want that? -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Com

[HACKERS] Small PATCH: check of 2 Perl modules

2016-02-12 Thread Eugene Kazakov
TAP-tests need two Perl modules: Test::More and IPC::Run. The patch adds checking of modules in configure.in and configure. Eugene Kazakov, Postgres Professional diff --git a/configure b/configure index 1903815..da6ed4b 100755 --- a/configure +++ b/configure @@ -15554,6 +15554,8 @@ done if