Re: [HACKERS] moving from contrib to bin

2015-04-22 Thread Bruce Momjian
On Wed, Apr 22, 2015 at 09:18:40AM +0200, Andres Freund wrote: > Peter, Michael, > > On 2015-04-22 16:13:15 +0900, Michael Paquier wrote: > > All the patches have been committed, finishing the work on this thread. > > Many thanks for that effort! And pg_upgrade thanks you. ;-) -- Bruce Momj

Re: [HACKERS] moving from contrib to bin

2015-04-22 Thread Andres Freund
Peter, Michael, On 2015-04-22 16:13:15 +0900, Michael Paquier wrote: > All the patches have been committed, finishing the work on this thread. Many thanks for that effort! Andres -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http:/

Re: [HACKERS] moving from contrib to bin

2015-04-22 Thread Michael Paquier
On Sun, Apr 12, 2015 at 12:36 PM, Peter Eisentraut wrote: > On 3/11/15 8:21 PM, Michael Paquier wrote: >> Attached is a series of patch rebased on current HEAD, there were some >> conflicts after perl-tidying the refactoring patch for MSVC. Note that >> this series still uses PGXS in the Makefiles

Re: [HACKERS] moving from contrib to bin

2015-04-15 Thread Peter Eisentraut
On 4/12/15 8:00 PM, Alvaro Herrera wrote: > Peter Eisentraut wrote: >> On 3/11/15 8:21 PM, Michael Paquier wrote: >>> Attached is a series of patch rebased on current HEAD, there were some >>> conflicts after perl-tidying the refactoring patch for MSVC. Note that >>> this series still uses PGXS in

Re: [HACKERS] moving from contrib to bin

2015-04-12 Thread Alvaro Herrera
Peter Eisentraut wrote: > On 3/11/15 8:21 PM, Michael Paquier wrote: > > Attached is a series of patch rebased on current HEAD, there were some > > conflicts after perl-tidying the refactoring patch for MSVC. Note that > > this series still uses PGXS in the Makefiles, I am fine to update them > > i

Re: [HACKERS] moving from contrib to bin

2015-04-11 Thread Peter Eisentraut
On 3/11/15 8:21 PM, Michael Paquier wrote: > Attached is a series of patch rebased on current HEAD, there were some > conflicts after perl-tidying the refactoring patch for MSVC. Note that > this series still uses PGXS in the Makefiles, I am fine to update them > if necessary once this matter is se

Re: [HACKERS] moving from contrib to bin

2015-03-11 Thread Michael Paquier
On Thu, Mar 12, 2015 at 8:50 AM, Peter Eisentraut wrote: > On 3/11/15 10:00 AM, Andres Freund wrote: >> On 2015-03-10 22:06:37 -0300, Alvaro Herrera wrote: >>> I don't think we care one bit whether these modules use pgxs, at least >>> not currently. If we find any issues later on, it should be an

Re: [HACKERS] moving from contrib to bin

2015-03-11 Thread Peter Eisentraut
On 3/11/15 10:00 AM, Andres Freund wrote: > On 2015-03-10 22:06:37 -0300, Alvaro Herrera wrote: >> I don't think we care one bit whether these modules use pgxs, at least >> not currently. If we find any issues later on, it should be an easy fix >> anyway. > > I personally find it quite ugly to us

Re: [HACKERS] moving from contrib to bin

2015-03-11 Thread Andres Freund
On 2015-03-11 11:19:24 -0300, Alvaro Herrera wrote: > Andres Freund wrote: > > On 2015-03-10 22:06:37 -0300, Alvaro Herrera wrote: > > > I don't think we care one bit whether these modules use pgxs, at least > > > not currently. If we find any issues later on, it should be an easy fix > > > anyway

Re: [HACKERS] moving from contrib to bin

2015-03-11 Thread Alvaro Herrera
Andres Freund wrote: > On 2015-03-10 22:06:37 -0300, Alvaro Herrera wrote: > > I don't think we care one bit whether these modules use pgxs, at least > > not currently. If we find any issues later on, it should be an easy fix > > anyway. > > I personally find it quite ugly to use pgxs for stuff i

Re: [HACKERS] moving from contrib to bin

2015-03-11 Thread Tom Lane
Andres Freund writes: > On 2015-03-10 22:06:37 -0300, Alvaro Herrera wrote: >> I don't think we care one bit whether these modules use pgxs, at least >> not currently. If we find any issues later on, it should be an easy fix >> anyway. > I personally find it quite ugly to use pgxs for stuff in >

Re: [HACKERS] moving from contrib to bin

2015-03-11 Thread Andres Freund
On 2015-03-10 22:06:37 -0300, Alvaro Herrera wrote: > I don't think we care one bit whether these modules use pgxs, at least > not currently. If we find any issues later on, it should be an easy fix > anyway. I personally find it quite ugly to use pgxs for stuff in src/bin. pgxs.mk says: # This f

Re: [HACKERS] moving from contrib to bin

2015-03-11 Thread Alvaro Herrera
Michael Paquier wrote: > On Tue, Mar 10, 2015 at 8:07 PM, Michael Paquier wrote: > > I'd rather vote for having the Windows-side stuff integrated with each > > patch. Mind if I rebase what you just sent with the Windows things > > added? > > And here is the rebased series with the MSVC changes inc

Re: [HACKERS] moving from contrib to bin

2015-03-10 Thread Michael Paquier
On Tue, Mar 10, 2015 at 8:07 PM, Michael Paquier wrote: > I'd rather vote for having the Windows-side stuff integrated with each > patch. Mind if I rebase what you just sent with the Windows things > added? And here is the rebased series with the MSVC changes included for each module in its indivi

Re: [HACKERS] moving from contrib to bin

2015-03-10 Thread Michael Paquier
On Wed, Mar 11, 2015 at 12:00 PM, Peter Eisentraut wrote: > Here is a rebase of my previous patch set. I have integrated the > various minor fixes from Michael and Andres. I have dropped moving > pg_standby, because no one seemed to like that. > > I wasn't able to do anything with Michael's Mkvc

Re: [HACKERS] moving from contrib to bin

2015-03-10 Thread Peter Eisentraut
Here is a rebase of my previous patch set. I have integrated the various minor fixes from Michael and Andres. I have dropped moving pg_standby, because no one seemed to like that. I wasn't able to do anything with Michael's Mkvcbuild.pm patch, since that appeared to include a significant refacto

Re: [HACKERS] moving from contrib to bin

2015-03-10 Thread Alvaro Herrera
Michael Paquier wrote: > On Wed, Mar 11, 2015 at 10:06 AM, Alvaro Herrera > wrote: > >> Now, the pushing plan was to have the stuff of pg_upgrade done in a > >> separate commit. Note that I am fine helping out wrapping up things > >> particularly on Windows if that helps. > > > > I vote for movin

Re: [HACKERS] moving from contrib to bin

2015-03-10 Thread Michael Paquier
On Wed, Mar 11, 2015 at 10:06 AM, Alvaro Herrera wrote: > Michael Paquier wrote: >> On Tue, Mar 10, 2015 at 10:48 PM, Alvaro Herrera >> wrote: > >> > How do we go about getting these patches pushed? >> >> I think that one of the last point raised (by Andres) was if the >> Makefiles in src/bin sho

Re: [HACKERS] moving from contrib to bin

2015-03-10 Thread Alvaro Herrera
Michael Paquier wrote: > On Tue, Mar 10, 2015 at 10:48 PM, Alvaro Herrera > wrote: > > How do we go about getting these patches pushed? > > I think that one of the last point raised (by Andres) was if the > Makefiles in src/bin should depend on pgxs.mk or not. FWIW, I think as > well that it sho

Re: [HACKERS] moving from contrib to bin

2015-03-10 Thread Michael Paquier
On Tue, Mar 10, 2015 at 10:48 PM, Alvaro Herrera wrote: > Andres Freund wrote: > >> Rebased (fair amount of trivial conflicts due to the copyright year >> bump) and attached as -MC style format-patch. If you look at the content >> of the patches you can see that the diff makes more sense now. > >

Re: [HACKERS] moving from contrib to bin

2015-03-10 Thread Alvaro Herrera
Andres Freund wrote: > Rebased (fair amount of trivial conflicts due to the copyright year > bump) and attached as -MC style format-patch. If you look at the content > of the patches you can see that the diff makes more sense now. Ah --- I just realized that the change Peter is proposing from bac

Re: [HACKERS] moving from contrib to bin

2015-01-21 Thread Bruce Momjian
On Sat, Jan 17, 2015 at 02:08:34PM +0100, Andres Freund wrote: > 7) Are we sure that the authors in the affected contrib modules are ok >with their authorship notice being removed? I don't think Ants, Bruce >or Simon have a problem with that, but ... I am fine. It means others can be blam

Re: [HACKERS] moving from contrib to bin

2015-01-21 Thread Bruce Momjian
On Sat, Jan 17, 2015 at 01:16:18PM +0100, Andres Freund wrote: > Hi, > > FWIW, I find it rather annoying if people attach patchsets as > tarballs. That makes it impossible to look at them in the mailreader > since I really don't have anything reasonable to go on to teach it to > treat it as a set

Re: [HACKERS] moving from contrib to bin

2015-01-18 Thread Michael Paquier
On Sun, Jan 18, 2015 at 10:21 PM, Andres Freund wrote: > On 2015-01-18 21:05:29 +0900, Michael Paquier wrote: >> On Sat, Jan 17, 2015 at 10:08 PM, Andres Freund >> wrote: >> > Observations: >> > 1) Are we sure it's a good idea to rely on pgxs.mk in src/bin programs? > >> Yeah, this seems like a

Re: [HACKERS] moving from contrib to bin

2015-01-18 Thread Peter Eisentraut
On 1/18/15 8:16 AM, Andres Freund wrote: > On 2015-01-18 22:02:13 +0900, Michael Paquier wrote: >> On Sun, Jan 18, 2015 at 9:56 PM, Andrew Gierth >>> 2) in any event it is essentially deprecated in favour of standby_mode >>> and therefore moving it to src/bin seems like a wrong move on two counts?

Re: [HACKERS] moving from contrib to bin

2015-01-18 Thread Peter Eisentraut
On 1/17/15 8:08 AM, Andres Freund wrote: > 1) Are we sure it's a good idea to rely on pgxs.mk in src/bin programs? I am. Why not? -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] moving from contrib to bin

2015-01-18 Thread Alvaro Herrera
Andres Freund wrote: > Hi, > > On 2015-01-18 12:56:59 +, Andrew Gierth wrote: > > and therefore moving it to src/bin seems like a wrong move on two counts? > > I personally agree that that pg_standby shouldn't be moved. Even if we > could not find agreement to remove it, there's little reaso

Re: [HACKERS] moving from contrib to bin

2015-01-18 Thread Andres Freund
On 2015-01-18 21:05:29 +0900, Michael Paquier wrote: > On Sat, Jan 17, 2015 at 10:08 PM, Andres Freund > wrote: > > Observations: > > 1) Are we sure it's a good idea to rely on pgxs.mk in src/bin programs? > Yeah, this seems like a bad dependency, PGXS being made for contrib > modules... So corr

Re: [HACKERS] moving from contrib to bin

2015-01-18 Thread Andres Freund
On 2015-01-18 22:02:13 +0900, Michael Paquier wrote: > On Sun, Jan 18, 2015 at 9:56 PM, Andrew Gierth > > 2) in any event it is essentially deprecated in favour of standby_mode > > and therefore moving it to src/bin seems like a wrong move on two counts? > There were some discussions about that a c

Re: [HACKERS] moving from contrib to bin

2015-01-18 Thread Andres Freund
Hi, On 2015-01-18 12:56:59 +, Andrew Gierth wrote: > Correct me if I'm wrong, but is it not the case that: > 1) pg_standby was intended to be customizable code, even if usable as > distributed, and I don't think that really happened in reality though... > 2) in any event it is essentially d

Re: [HACKERS] moving from contrib to bin

2015-01-18 Thread Michael Paquier
On Sun, Jan 18, 2015 at 9:56 PM, Andrew Gierth wrote: > Correct me if I'm wrong, but is it not the case that: > 1) pg_standby was intended to be customizable code, even if usable as > distributed, and I am not sure about that. > 2) in any event it is essentially deprecated in favour of standby_mo

Re: [HACKERS] moving from contrib to bin

2015-01-18 Thread Andrew Gierth
Correct me if I'm wrong, but is it not the case that: 1) pg_standby was intended to be customizable code, even if usable as distributed, and 2) in any event it is essentially deprecated in favour of standby_mode and therefore moving it to src/bin seems like a wrong move on two counts? -- Andre

Re: [HACKERS] moving from contrib to bin

2015-01-18 Thread Michael Paquier
Simon, Bruce, Ants, (CCing..) On Sun, Jan 18, 2015 at 9:05 PM, Michael Paquier wrote: > On Sat, Jan 17, 2015 at 10:08 PM, Andres Freund > wrote: >> 7) Are we sure that the authors in the affected contrib modules are ok >>with their authorship notice being removed? I don't think Ants, Bruce

Re: [HACKERS] moving from contrib to bin

2015-01-18 Thread Michael Paquier
On Sat, Jan 17, 2015 at 10:08 PM, Andres Freund wrote: > Observations: > 1) Are we sure it's a good idea to rely on pgxs.mk in src/bin programs? Yeah, this seems like a bad dependency, PGXS being made for contrib modules... So corrected in the patch attached (the headers of the Makefiles are impro

Re: [HACKERS] moving from contrib to bin

2015-01-17 Thread Andres Freund
On 2015-01-17 13:16:18 +0100, Andres Freund wrote: > I'd also like to see patches that primarily move code around as git diff > -M -C style diffs (can also be passed to format-patch). That will show > the file move and then additionally the changes that have been made in > addition to the rename. T

Re: [HACKERS] moving from contrib to bin

2015-01-17 Thread Andres Freund
Hi, FWIW, I find it rather annoying if people attach patchsets as tarballs. That makes it impossible to look at them in the mailreader since I really don't have anything reasonable to go on to teach it to treat it as a set of patches. I'd also like to see patches that primarily move code around a

Re: [HACKERS] moving from contrib to bin

2015-01-14 Thread Michael Paquier
On Tue, Dec 23, 2014 at 3:24 PM, Michael Paquier wrote: > On Mon, Dec 22, 2014 at 11:30 PM, Alvaro Herrera > wrote: >> Michael Paquier wrote: >> >>> And here is an updated patch following those lines. Similarly to the >>> things in contrib/, a set of variables are used to define for each >>> modu

Re: [HACKERS] moving from contrib to bin

2014-12-22 Thread Alvaro Herrera
Michael Paquier wrote: > And here is an updated patch following those lines. Similarly to the > things in contrib/, a set of variables are used to define for each > module what are the extra libraries, include dirs, etc. This refactors > quite a bit of code, even if there are a couple of exception

Re: [HACKERS] moving from contrib to bin

2014-12-22 Thread Michael Paquier
On Thu, Dec 18, 2014 at 10:37 AM, Michael Paquier wrote: > On Thu, Dec 18, 2014 at 4:02 AM, Alvaro Herrera > wrote: >> >> I know this is how it currently works, but it looks way too messy to me: >> >> + my $pgarchivecleanup = AddSimpleFrontend('pg_archivecleanup'); >> + my $pgstandby

Re: [HACKERS] moving from contrib to bin

2014-12-17 Thread Michael Paquier
On Thu, Dec 18, 2014 at 4:02 AM, Alvaro Herrera wrote: > > I know this is how it currently works, but it looks way too messy to me: > > + my $pgarchivecleanup = AddSimpleFrontend('pg_archivecleanup'); > + my $pgstandby = AddSimpleFrontend('pg_standby'); > + my $pgtestfsync = AddS

Re: [HACKERS] moving from contrib to bin

2014-12-17 Thread Alvaro Herrera
I know this is how it currently works, but it looks way too messy to me: + my $pgarchivecleanup = AddSimpleFrontend('pg_archivecleanup'); + my $pgstandby = AddSimpleFrontend('pg_standby'); + my $pgtestfsync = AddSimpleFrontend('pg_test_fsync'); + my $pgtesttiming = AddSimpl

Re: [HACKERS] moving from contrib to bin

2014-12-15 Thread Michael Paquier
On Mon, Dec 15, 2014 at 11:53 AM, Michael Paquier wrote: > On Mon, Dec 15, 2014 at 11:45 AM, Peter Eisentraut wrote: >> On 12/14/14 9:08 PM, Michael Paquier wrote: - no Windows build system support yet >>> Do you need some help here? >> Sure. Peter, Attached is a patch that can be applied o

Re: [HACKERS] moving from contrib to bin

2014-12-14 Thread Michael Paquier
On Mon, Dec 15, 2014 at 11:45 AM, Peter Eisentraut wrote: > On 12/14/14 9:08 PM, Michael Paquier wrote: >>> - no Windows build system support yet >> Do you need some help here? > > Sure. > >> I am getting worried about potential >> breakage with the version information built with MSVC and MinGW..

Re: [HACKERS] moving from contrib to bin

2014-12-14 Thread Peter Eisentraut
On 12/14/14 9:08 PM, Michael Paquier wrote: >> - no Windows build system support yet > Do you need some help here? Sure. > I am getting worried about potential > breakage with the version information built with MSVC and MinGW.. I don't know what that is. -- Sent via pgsql-hackers mailing l

Re: [HACKERS] moving from contrib to bin

2014-12-14 Thread Michael Paquier
On Mon, Dec 15, 2014 at 10:59 AM, Peter Eisentraut wrote: > - no Windows build system support yet Do you need some help here? I am getting worried about potential breakage with the version information built with MSVC and MinGW.. > - We have traditionally kept an individual author acknowledgement

Re: [HACKERS] moving from contrib to bin

2014-12-14 Thread Peter Eisentraut
Here is a patch series to move pg_archivecleanup pg_standby pg_test_fsync pg_test_timing pg_upgrade pg_xlogdump pgbench from contrib/ to src/bin/. Move people didn't like moving oid2name and vacuumlo, which I understand. So I'll leave those for now. I have also included a patch to move pg_upgr

Re: [HACKERS] moving from contrib to bin

2014-12-13 Thread Christoph Berg
Re: Alvaro Herrera 2014-12-12 <20141212203700.gb1...@alvh.no-ip.org> > > Pardon me for not knowing much about Debian packages, but how would > > that work exactly? Is it possible to do make install-client, then > > package the installed files, then rm -rf the install tree, then > > repeat for inst

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Bruce Momjian
On Fri, Dec 12, 2014 at 08:57:55PM -0500, Tom Lane wrote: > Peter Eisentraut writes: > > On 12/12/14 10:16 AM, Tom Lane wrote: > >> I think pg_upgrade should continue to have SQL scripts that create and > >> delete the SQL function definitions for these. > > > That won't actually work very easily

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Peter Eisentraut
On 12/12/14 11:14 AM, Robert Haas wrote: > contrib is a nice middle-ground. I respect that view. But if we consider contrib a place where things can try to mature, then there must be a way to move things out of there once they have matured. If it turns out that moving things out of contrib cause

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Peter Eisentraut
On 12/12/14 11:45 AM, Andres Freund wrote: > On 2014-12-12 11:42:57 -0500, Robert Haas wrote: >>> I think the amount of effort a simple renamed directory which wholly >>> contains a binary creates is acceptable. Just use patch -p4 instead of >>> patch -p1... >> >> That is fine if you are manually a

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Tom Lane
Peter Eisentraut writes: > On 12/12/14 10:16 AM, Tom Lane wrote: >> I think pg_upgrade should continue to have SQL scripts that create and >> delete the SQL function definitions for these. > That won't actually work very easily. LANGUAGE internal functions need > to be in fmgr_builtins, and the

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Peter Eisentraut
On 12/12/14 3:20 PM, Tom Lane wrote: > Pardon me for not knowing much about Debian packages, but how would > that work exactly? Is it possible to do make install-client, then > package the installed files, then rm -rf the install tree, then > repeat for install-server and install-contrib? In the

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Peter Eisentraut
On 12/12/14 10:16 AM, Tom Lane wrote: > I don't particularly object to having the C code built into the backend; > there's not that much of it, and if we could static-ize some of the global > variables that are involved presently, it'd be a Good Thing IMO. However, > the current arrangement makes

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Michael Paquier
On Sat, Dec 13, 2014 at 1:00 AM, Alvaro Herrera wrote: > Tom Lane wrote: >> Heikki Linnakangas writes: >> > On 12/12/2014 03:07 PM, Peter Eisentraut wrote: >> >> On 12/9/14 4:10 PM, Alvaro Herrera wrote: >> >>> Maybe it makes sense to have a distinction between client programs and >> >>> server p

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Robert Haas
On Fri, Dec 12, 2014 at 12:19 PM, Alvaro Herrera wrote: >> If we decide that executables can no longer live in contrib, > > Nobody is saying that. The reason I though that might be on the table is that the first post on this thread, at least as I understood it, proposed to move every executable o

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Alvaro Herrera
Tom Lane wrote: > Christoph Berg writes: > > However, for PostgreSQL this means lengthy debian/*.install files > > (the equivalent of %files in rpm spec speak): > > Right ... > > > If there were separate "install-client", "install-server", and > > "install-contrib" targets, that would probably s

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Tom Lane
Christoph Berg writes: > However, for PostgreSQL this means lengthy debian/*.install files > (the equivalent of %files in rpm spec speak): Right ... > If there were separate "install-client", "install-server", and > "install-contrib" targets, that would probably shorten those files > quite a bit

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Christoph Berg
Re: Andres Freund 2014-12-12 <20141212152723.go31...@awork2.anarazel.de> > On 2014-12-12 10:20:58 -0500, Tom Lane wrote: > > Peter Eisentraut writes: > > > On 12/12/14 8:13 AM, Andres Freund wrote: > > >> Wouldn't a make install-server/client targets or something similar > > >> actually achieve th

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Alvaro Herrera
Robert Haas wrote: > On Fri, Dec 12, 2014 at 11:00 AM, Alvaro Herrera > wrote: > > So let's put the whole bunch under src/bin/ and be done with it. > > I'm not really convinced this is a very good idea. What do we get out > of moving everything, or even anything, from contrib? We show that it'

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Bruce Momjian
On Fri, Dec 12, 2014 at 10:16:05AM -0500, Tom Lane wrote: > Peter Eisentraut writes: > > On 12/9/14 4:32 PM, Bruce Momjian wrote: > >> On Tue, Dec 9, 2014 at 06:10:02PM -0300, Alvaro Herrera wrote: > >>> (For pg_upgrade you also need to do something about pg_upgrade_support, > >>> which is good b

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Tom Lane
Andres Freund writes: > On 2014-12-12 11:42:57 -0500, Robert Haas wrote: >> And I don't know how well git cherry-pick will follow >> the moves. > Not well if the patch is done in master first. FWIW, I always patch master first, and have zero intention of changing that workflow. (I have given re

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Andres Freund
On 2014-12-12 11:42:57 -0500, Robert Haas wrote: > > I think the amount of effort a simple renamed directory which wholly > > contains a binary creates is acceptable. Just use patch -p4 instead of > > patch -p1... > > That is fine if you are manually applying a patch that touches only > that direc

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Robert Haas
On Fri, Dec 12, 2014 at 11:40 AM, Andres Freund wrote: > The benefit of moving relevant stuff is that it'll actually be installed > by default when installing postgres on many platforms. That's currently > often not the case. The contrib umbrella, as used by many other > projects, actually justifi

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Robert Haas
On Fri, Dec 12, 2014 at 11:26 AM, Tom Lane wrote: > Robert Haas writes: >> I'm not really convinced this is a very good idea. What do we get out >> of moving everything, or even anything, from contrib? It will make >> back-patching harder, but more importantly, it will possibly create >> the fa

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Andres Freund
On 2014-12-12 11:14:56 -0500, Robert Haas wrote: > I'm not really convinced this is a very good idea. What do we get out > of moving everything, or even anything, from contrib? The benefit of moving relevant stuff is that it'll actually be installed by default when installing postgres on many pla

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Tom Lane
Robert Haas writes: > I'm not really convinced this is a very good idea. What do we get out > of moving everything, or even anything, from contrib? It will make > back-patching harder, but more importantly, it will possibly create > the false impression that everything we distribute is on equal

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Joshua D. Drake
On 12/08/2014 07:50 PM, Tom Lane wrote: Peter Eisentraut writes: Last time this was attempted, the discussion got lost in exactly which extensions are worthy enough to be considered official or something like that. I want to dodge that here by starting at the opposite end: 1. move programs to

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Robert Haas
On Fri, Dec 12, 2014 at 11:00 AM, Alvaro Herrera wrote: >> I'm pretty much -1 on relocating anything that's under src/bin already. I agree. I can't see packagers putting it anywhere except for $SOMETHING/bin in the final install, so what do we get out of dividing it up in some weird way in our t

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Alvaro Herrera
Tom Lane wrote: > Heikki Linnakangas writes: > > On 12/12/2014 03:07 PM, Peter Eisentraut wrote: > >> On 12/9/14 4:10 PM, Alvaro Herrera wrote: > >>> Maybe it makes sense to have a distinction between client programs and > >>> server programs. Can we have src/sbin/ and move stuff that involves th

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Andres Freund
On 2014-12-12 10:20:58 -0500, Tom Lane wrote: > Peter Eisentraut writes: > > On 12/12/14 8:13 AM, Andres Freund wrote: > >> Wouldn't a make install-server/client targets or something similar > >> actually achieve the same thing? Seems simpler to maintain to me. > > > Adding non-standard makefile

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Tom Lane
Peter Eisentraut writes: > On 12/12/14 8:13 AM, Andres Freund wrote: >> Wouldn't a make install-server/client targets or something similar >> actually achieve the same thing? Seems simpler to maintain to me. > Adding non-standard makefile targets comes with its own set of > maintenance issues. I

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Tom Lane
Peter Eisentraut writes: > On 12/9/14 4:32 PM, Bruce Momjian wrote: >> On Tue, Dec 9, 2014 at 06:10:02PM -0300, Alvaro Herrera wrote: >>> (For pg_upgrade you also need to do something about pg_upgrade_support, >>> which is good because that is one very ugly crock.) >> FYI, pg_upgrade_support was

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Tom Lane
Heikki Linnakangas writes: > On 12/12/2014 03:07 PM, Peter Eisentraut wrote: >> On 12/9/14 4:10 PM, Alvaro Herrera wrote: >>> Maybe it makes sense to have a distinction between client programs and >>> server programs. Can we have src/sbin/ and move stuff that involves the >>> server side in there

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Andres Freund
On 2014-12-12 11:27:01 -0300, Alvaro Herrera wrote: > We already have src/bin/; the mixture of "src/" and "bin/" predates us. > Of course, the stuff we keep in there is not binaries but source code > that produces binaries. > > As for src/sbin/, we wouldn't install anything to the system's > /usr/

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Alvaro Herrera
Bruce Momjian wrote: > On Fri, Dec 12, 2014 at 03:13:44PM +0200, Heikki Linnakangas wrote: > > On 12/12/2014 03:11 PM, Heikki Linnakangas wrote: > > >Sounds good. We already separate server and client programs in the docs, > > >and packagers put them in different packages too. This should make > >

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Peter Eisentraut
On 12/12/14 8:13 AM, Andres Freund wrote: > Wouldn't a make install-server/client targets or something similar > actually achieve the same thing? Seems simpler to maintain to me. Adding non-standard makefile targets comes with its own set of maintenance issues. Restructuring the source tree and h

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Peter Eisentraut
On 12/8/14 10:50 PM, Tom Lane wrote: > oid2name and vacuumlo, besides being of very dubious general utility, > are fails from a namespacing standpoint. If we were to promote them > into standard install components I think a minimum requirement should be > to rename them to pg_something. (oid2name

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Bruce Momjian
On Fri, Dec 12, 2014 at 08:11:31AM -0500, Peter Eisentraut wrote: > On 12/9/14 4:32 PM, Bruce Momjian wrote: > > On Tue, Dec 9, 2014 at 06:10:02PM -0300, Alvaro Herrera wrote: > >> (For pg_upgrade you also need to do something about pg_upgrade_support, > >> which is good because that is one very u

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Bruce Momjian
On Fri, Dec 12, 2014 at 03:13:44PM +0200, Heikki Linnakangas wrote: > On 12/12/2014 03:11 PM, Heikki Linnakangas wrote: > >On 12/12/2014 03:07 PM, Peter Eisentraut wrote: > >>On 12/9/14 4:10 PM, Alvaro Herrera wrote: > >>>Maybe it makes sense to have a distinction between client programs and > >>>s

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Heikki Linnakangas
On 12/12/2014 03:11 PM, Heikki Linnakangas wrote: On 12/12/2014 03:07 PM, Peter Eisentraut wrote: On 12/9/14 4:10 PM, Alvaro Herrera wrote: Maybe it makes sense to have a distinction between client programs and server programs. Can we have src/sbin/ and move stuff that involves the server side

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Andres Freund
On 2014-12-12 15:11:01 +0200, Heikki Linnakangas wrote: > On 12/12/2014 03:07 PM, Peter Eisentraut wrote: > >On 12/9/14 4:10 PM, Alvaro Herrera wrote: > >>Maybe it makes sense to have a distinction between client programs and > >>server programs. Can we have src/sbin/ and move stuff that involves

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Heikki Linnakangas
On 12/12/2014 03:07 PM, Peter Eisentraut wrote: On 12/9/14 4:10 PM, Alvaro Herrera wrote: Maybe it makes sense to have a distinction between client programs and server programs. Can we have src/sbin/ and move stuff that involves the server side in there? I think that'd be pg_xlogdump, pg_archi

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Peter Eisentraut
On 12/9/14 4:32 PM, Bruce Momjian wrote: > On Tue, Dec 9, 2014 at 06:10:02PM -0300, Alvaro Herrera wrote: >> (For pg_upgrade you also need to do something about pg_upgrade_support, >> which is good because that is one very ugly crock.) > > FYI, pg_upgrade_support was segregated from pg_upgrade on

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Peter Eisentraut
On 12/9/14 4:10 PM, Alvaro Herrera wrote: > Maybe it makes sense to have a distinction between client programs and > server programs. Can we have src/sbin/ and move stuff that involves the > server side in there? I think that'd be pg_xlogdump, pg_archivecleanup, > pg_upgrade, pg_test_timing, pg_t

Re: [HACKERS] moving from contrib to bin

2014-12-12 Thread Peter Eisentraut
On 12/9/14 1:18 PM, Josh Berkus wrote: > The one exception I might make above is pg_standby. What do we need > this for today, exactly? This was discussed recently and people wanted to keep it. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subsc

Re: [HACKERS] moving from contrib to bin

2014-12-09 Thread Bruce Momjian
On Tue, Dec 9, 2014 at 06:10:02PM -0300, Alvaro Herrera wrote: > (For pg_upgrade you also need to do something about pg_upgrade_support, > which is good because that is one very ugly crock.) FYI, pg_upgrade_support was segregated from pg_upgrade only because we wanted separate binary and shared o

Re: [HACKERS] moving from contrib to bin

2014-12-09 Thread Alvaro Herrera
Peter Eisentraut wrote: > Here are the contrib programs: > > oid2name > pg_archivecleanup > pg_standby > pg_test_fsync > pg_test_timing > pg_upgrade > pg_xlogdump > pgbench > vacuumlo > > The proposal would basically be to mv contrib/$x src/bin/$x and also > move the reference pages in the docum

Re: [HACKERS] moving from contrib to bin

2014-12-09 Thread Robert Haas
On Mon, Dec 8, 2014 at 10:26 PM, Peter Eisentraut wrote: > Let's take another crack at moving stuff out of contrib. Nobody likes > contrib. The task is only finding something that most people like better. I like contrib fine. It's a great place for things to live that are not quite baked enoug

Re: [HACKERS] moving from contrib to bin

2014-12-09 Thread Josh Berkus
> Here are the contrib programs: > > oid2name > pg_archivecleanup > pg_standby > pg_test_fsync > pg_test_timing > pg_upgrade > pg_xlogdump > pgbench > vacuumlo > > The proposal would basically be to mv contrib/$x src/bin/$x and also > move the reference pages in the documentation. +1 Consideri

Re: [HACKERS] moving from contrib to bin

2014-12-08 Thread Peter Geoghegan
On Mon, Dec 8, 2014 at 9:00 PM, Andres Freund wrote: > I actually think both are quite useful when setting up new systems to > quickly screen for problems. There still is a fairly large number of > virtualized systems with pretty much broken timing functions; and > checking whether fsync actually

Re: [HACKERS] moving from contrib to bin

2014-12-08 Thread Andres Freund
On 2014-12-08 22:50:30 -0500, Tom Lane wrote: > I'm not exactly convinced that we want to encourage packagers to include > either pg_test_fsync or pg_test_timing in standard packages. They are not > all that useful to ordinary users. I actually think both are quite useful when setting up new syst

Re: [HACKERS] moving from contrib to bin

2014-12-08 Thread Tom Lane
Peter Eisentraut writes: > Last time this was attempted, the discussion got lost in exactly which > extensions are worthy enough to be considered official or something like > that. I want to dodge that here by starting at the opposite end: > 1. move programs to src/bin/ > Here are the contrib pr