Re: Ready for beta yet? - Mailing list pgadmin-hackers

From Tomasz Rybak
Subject Re: Ready for beta yet?
Date
Msg-id 1128453873.4642.21.camel@localhost.localdomain
Whole thread Raw
In response to Re: Ready for beta yet?  (blacknoz@club-internet.fr)
List pgadmin-hackers
Dnia 03-10-2005, pon o godzinie 18:50 +0200, blacknoz@club-internet.fr
napisał(a):
> Hi Tomasz, Dave and friends,
>
> first day to my new job and first day with an access to
> the net since a long time.

Good luck with new job.

> Tomasz, if you have some time to produce a merge between
> what you provided and what I've fixed in official debian I'd
> appreciate your help now I particularly think to the pgagent part
> of the package which was not taken in consideration in 1.2.2 and
> also the i18n relocation.
>

OK.
Here are changes I made, applied to revision 4490.
They are mostly changes made by you in official Debian package,
but I made few additional.

For changelog, I put 1.4.0, because as I understand, we're trying
to be ready for 1.4 release; I also put you name, as author of these
changes. Feel free to put mine, if it's more appropriate.

In rules, line 16, instead of
_pgsql_inc:="/usr/include/postgresql -I./include"
I put calling of pg_config; such behaviour was mentioned
in changelog for libpq-dev 8.0.3-13 as more proper now.

I also created new variable CPPFLAGS, where I put -I./include
taken from _pgsql_inc.
Previous situation resulted in warnings from configure script,
because instead of putting -I into _pgsql_inc, it was passed
to script as parameter, which wasn't sure what to do with that.
pgAdmin was being built successfully, but I decided to get
rid of this warning.
I also had to change a bit calling configure script in line 50,
and add CPPFLAGS to it.

I also changed directory from ui to i18n line 106 of file rules.

Last change I made is adding pgagent and i18n files in pgadmin3.install.

Changelog in attachment.

One remark.
Why there are slony3 and slony3-data package?
Both depend on each other, and from my point of view there
is no need for them both; maybe it's good idea to merge them.
However, I'm not experienced in Debian packages creating,
and I don't know why split occurred, so I'll leave these two
as they are, without changes. So it's up to you to decide what
to do.

--
Tomasz Rybak <bogomips@post.pl>

Attachment

pgadmin-hackers by date:

Previous
From: svn@pgadmin.org
Date:
Subject: SVN Commit by andreas: r4490 - trunk/www/pgadmin3/cnt
Next
From: Andreas Pflug
Date:
Subject: Re: Ready for beta yet?