Great. Thanks.
On Sun, Dec 15, 2013 at 3:15 AM, Steve Singer <[email protected]>wrote: > On 12/13/2013 12:31 AM, Sandeep Thakkar wrote: > >> Yes, the patch looks fine. >> >> > This has been committed to > master, REL_2_2_STABLE and REL_2_1_STABLE > > > >> On Thu, Dec 12, 2013 at 8:01 PM, Steve Singer <[email protected] >> <mailto:[email protected]>> wrote: >> >> On 12/12/2013 03:49 AM, Sandeep Thakkar wrote: >> >> The development package >> postgresql92-devel-9.2.6-__1PGDG.rhel5.x86_64.rpm >> and postgresql93-devel-9.3.2-__1PGDG.rhel5.x86_64.rpm at >> http://yum.postgresql.org/__rpmchart.php >> >> <http://yum.postgresql.org/rpmchart.php> >> >> also has the port.h in the server include. >> >> >> libpq-dev seems to install port.h in /usr/include/postgresql >> >> root@debian:~/slony1-engine# dpkg-query -S >> /usr/include/postgresql/port.h >> libpq-dev: /usr/include/postgresql/port.h >> >> http://packages.debian.org/__sid/i386/libpq-dev/filelist >> >> <http://packages.debian.org/sid/i386/libpq-dev/filelist> >> >> The RPM packages don't port port.h there. >> >> How about the attached patch? We will add the server includes back >> in on pgport builds for slonik. >> >> >> >> >> >> >> On Thu, Dec 12, 2013 at 12:23 PM, Sandeep Thakkar >> <sandeep.thakkar@enterprisedb.__com >> <mailto:[email protected]> >> <mailto:sandeep.thakkar@__enterprisedb.com >> >> <mailto:[email protected]>>> wrote: >> >> I see the port.h also present in the postgresql-server-dev >> package. >> Here is the list: >> >> http://packages.debian.org/__sid/i386/postgresql-server-__ >> dev-9.3/filelist >> >> <http://packages.debian.org/sid/i386/postgresql-server- >> dev-9.3/filelist> >> >> if you build the sources, you will find port.h always in >> server include. >> >> >> On Wed, Dec 11, 2013 at 9:28 PM, Steve Singer >> <[email protected] <mailto:[email protected]> >> <mailto:[email protected].__info >> <mailto:[email protected]>>> wrote: >> >> On 12/11/2013 05:14 AM, Sandeep Thakkar wrote: >> >> How did you install postgresql on Debian? I have >> built >> PosrgreSQL >> sources on CentOS, OSX and Windows and the port.h is >> installed in >> include/postgresql/server/____port.h and >> include/postgresql/internal/____port.h >> >> on all platforms. >> >> >> >> The packages from apt.postgresql.org >> <http://apt.postgresql.org> <http://apt.postgresql.org> >> >> >> >> >> >> >> On Sun, Dec 8, 2013 at 9:40 AM, Steve Singer >> <[email protected] >> <mailto:[email protected]> >> <mailto:[email protected].__info <mailto:[email protected] >> >> >> <mailto:[email protected]. >> <mailto:[email protected].>____info >> <mailto:[email protected].__info >> <mailto:[email protected]>>>> wrote: >> >> On 11/29/2013 04:30 AM, Sandeep Thakkar wrote: >> >> The only work around I see now if to set the >> CPPFLAGS in the >> environment >> to the server include directory, so that >> CPPFLAGS_CLIENT takes >> it in the >> Makefile.global. >> >> >> >> So in debian (or at least with Wheezy) where the >> original bug 315 >> was reported >> >> >> libpq-dev puts port.h in >> /usr/include/postgresql/port.h >> so it gets >> picked up by the client include paths. I am >> able to do >> a build with >> --with-pgport >> >> This isn't actually a problem on debian, is >> the problem >> on OSX that >> port.h isn't being put in the libpq header >> directories? >> Should it? >> Where do other platforms/distributions put >> port.h ? >> >> >> >> On Tue, Nov 26, 2013 at 7:05 PM, Steve >> Singer >> <[email protected] >> <mailto:[email protected]> >> <mailto:[email protected].__info >> <mailto:[email protected]>> >> <mailto:[email protected]. >> <mailto:[email protected].>____info >> <mailto:[email protected].__info >> <mailto:[email protected]>>> >> <mailto:[email protected] >> <mailto:[email protected]>. >> <mailto:[email protected] >> <mailto:[email protected]>.>______info >> >> >> >> <mailto:[email protected]. >> <mailto:[email protected].>____info >> <mailto:[email protected].__info >> <mailto:[email protected]>>>>> wrote: >> >> On 11/26/2013 04:14 AM, Sandeep >> Thakkar wrote: >> >> Hi >> >> I was building Slony1-2.2.1 >> sources on Mac >> OS X and >> found that >> the build >> fails because of: >> >> slonik.c:49:18: error: port.h: No >> such >> file or directory >> CPPFLAGS is now defined to just >> CPPFLAGS_CLIENT that >> does not >> include >> the PG "server" include directory >> >> ('<pginstall>/include/________postgresql/server' >> >> contains >> >> >> port.h). Any >> >> specific >> reason why this change was made >> in 2.2.1? >> >> >> >> http://www.slony.info/________bugzilla/show_bug.cgi?id=315 >> <http://www.slony.info/______bugzilla/show_bug.cgi?id=315> >> >> >> <http://www.slony.info/______bugzilla/show_bug.cgi?id=315 >> <http://www.slony.info/____bugzilla/show_bug.cgi?id=315>> >> >> >> >> <http://www.slony.info/______bugzilla/show_bug.cgi?id=315 >> <http://www.slony.info/____bugzilla/show_bug.cgi?id=315> >> >> <http://www.slony.info/____bugzilla/show_bug.cgi?id=315 >> <http://www.slony.info/__bugzilla/show_bug.cgi?id=315>>> >> >> >> >> >> <http://www.slony.info/______bugzilla/show_bug.cgi?id=315 >> <http://www.slony.info/____bugzilla/show_bug.cgi?id=315> >> >> <http://www.slony.info/____bugzilla/show_bug.cgi?id=315 >> <http://www.slony.info/__bugzilla/show_bug.cgi?id=315>> >> >> >> <http://www.slony.info/____bugzilla/show_bug.cgi?id=315 >> <http://www.slony.info/__bugzilla/show_bug.cgi?id=315> >> >> <http://www.slony.info/__bugzilla/show_bug.cgi?id=315 >> <http://www.slony.info/bugzilla/show_bug.cgi?id=315>>>__> >> >> >> On systems that have multiple versions >> of >> postgresql >> installed might >> pull in files for the client from one >> version >> and server >> includes >> from another. You used to be able to >> get away >> with this >> but in more >> recent versions of PG this breaks >> things. >> >> I am not exactly sure what the best >> way to >> deal with this >> in the >> PGPORT case? I have a feeling just >> adding the >> server >> includes back >> in on --with-pgport builds will get >> us back to >> the bug 315 >> situation >> >> >> >> >> >> -- >> Sandeep Thakkar >> >> >> >> >> _______________________________________________________ >> Slony1-general mailing list >> >> [email protected].________info >> <mailto:Slony1-general@lists >> <mailto:Slony1-general@lists>. >> <mailto:Slony1-general@lists >> <mailto:Slony1-general@lists>.>______slony.info >> <http://slony.info> <http://slony.info> >> >> <mailto:Slony1-general@lists. >> <mailto:Slony1-general@lists.>____slony.info <http://slony.info> >> <mailto:Slony1-general@lists.__slony.info >> <mailto:[email protected]>>>> >> http://lists.slony.info/________mailman/listinfo/slony1-____ >> ____general >> <http://lists.slony.info/______mailman/listinfo/slony1-_____ >> _general> >> >> >> <http://lists.slony.info/______mailman/listinfo/slony1-_____ >> _general >> <http://lists.slony.info/____mailman/listinfo/slony1-____general >> >> >> >> >> >> <http://lists.slony.info/______mailman/listinfo/slony1-_____ >> _general >> <http://lists.slony.info/____mailman/listinfo/slony1-____general> >> >> <http://lists.slony.info/____mailman/listinfo/slony1-____general >> <http://lists.slony.info/__mailman/listinfo/slony1-__general>>> >> >> >> >> >> >> <http://lists.slony.info/______mailman/listinfo/slony1-_____ >> _general >> <http://lists.slony.info/____mailman/listinfo/slony1-____general> >> >> <http://lists.slony.info/____mailman/listinfo/slony1-____general >> <http://lists.slony.info/__mailman/listinfo/slony1-__general>> >> >> >> <http://lists.slony.info/____mailman/listinfo/slony1-____general >> <http://lists.slony.info/__mailman/listinfo/slony1-__general> >> >> <http://lists.slony.info/__mailman/listinfo/slony1-__general >> <http://lists.slony.info/mailman/listinfo/slony1-general>>>> >> >> >> >> >> >> -- >> Sandeep Thakkar >> >> >> >> >> >> -- >> Sandeep Thakkar >> >> >> >> >> >> -- >> Sandeep Thakkar >> >> >> >> >> -- >> Sandeep Thakkar >> >> >> >> >> >> -- >> Sandeep Thakkar >> >> > -- Sandeep Thakkar
_______________________________________________ Slony1-general mailing list [email protected] http://lists.slony.info/mailman/listinfo/slony1-general
