lp:ubuntu/maverick/postgresql-common

Created by James Westby and last modified
Get this branch:
bzr branch lp:ubuntu/maverick/postgresql-common
Members of Ubuntu branches can upload to this branch. Log in for directions.

Related bugs

Related blueprints

Branch information

Owner:
Ubuntu branches
Review team:
Ubuntu Development Team
Status:
Mature

Recent revisions

21. By Martin Pitt

* Urgency high since this fixes two RC bugs.
* t/030_errors.t: Check that pg_createcluster leaves the original one intact
  if the cluster already exists, also when the original one is not running.
  This reproduces #597097.
* pg_createcluster: Be more careful with cleaning up the created cluster if
  an error occurs: Do not start the cleanup until we actually passed our
  sanity checks and created files for the new cluster. Before, it would
  erroneously remove an already existing cluster on a sanity check fail, if
  that cluster happened to not be running at the time. (Closes: #597097)
* debian/supported-versions: Be more robust against lsb_release failing, e.
  g. in the case where it is not fully configured yet. (Closes: #597561)
* debian/supported-versions: Drop check for /etc/debian_version if
  lsb_release is not working/existing. Derivatives have debian_version as
  well, and we don't actually evaluate it, so just print a meaningful error
  message and go with the default versions.
* debian/rules: Put init script priority back to S29/K21 to match the
  previous postgresql-8.4 init script. Fix the priorities on upgrade in
  debian/postgresql-common.preinst. (Closes: #597654)

20. By Martin Pitt

[ Martin Pitt ]
* t/080_start.conf.t: Update for common init.d script.
* debian/rules: Drop installation of lintian overrides and let dh_lintian do
  it for us. Adapt override file accordingly.
* debian/control: Drop unnecessary debconf dependency from -client-common.
* Bump Standards-Version to 3.9.1, no changes necessary.
* debian/postgresql-common.postgresql.init: Fix force-reload, thanks Pascal
  Volk. (Closes: #591185)
* debian/rules: Stop man page build/clean being package dependent.
* pg_buildext: Remove bashishms and convert to plain /bin/sh.

[ Dimitri Fontaine ]
* Add new package postgresql-server-dev-all:
  - "pb_buildext" tool for easy building of extension packages for multiple
    server versions.
  - pgxs_debian_control.mk debian/rules include for generating
    debian/control.

19. By Martin Pitt

* pg_lsclusters: Fix "uninitialized value" warning when a cluster's data
  directory is not set. This is a rather fatal problem anyway, but we can
  present it in a nicer way. Thanks Christoph Berg! (Closes: #589014)
* Migrate to a common init script for all server versions, to avoid
  providing the "postgresql" service in multiple packages (which causes
  insserv to complain bitterly):
  - debian/init.d-functions: Add function get_versions() which returns all
    installed versions, but filters out the ones which have their own init
    script. With that, we can run newer common versions with older server
    packages.
  - debian/maintscripts-functions: Add function stop_version() (to be called
    from "prerm upgrade"), and start cluster in configure_version() if there
    is no version specific init script.
  - Add debian/postgresql-common.postgresql.init: Common init script for all
    installed server versions.
  - debian/rules: Install common init script.
  - (Closes: #589524)
* Add debian/source/format: 3.0 (native).
* debian/control: Bump Standards-Version to 3.9.0 (no changes necessary).

18. By Martin Pitt

* debian/supported-versions: Drop 8.3 from squeeze/unstable. Add 9.0 to all
  Debian/Ubuntu versions to support backports.
* debian/README.Debian: Talk about 8.3 and 8.4, since those are the versions
  relevant to squeeze installs and upgrades.
* pg_upgradecluster POD: Describe allowed characters in upgrade script file
  names.
* debian/supported-versions: Add Ubuntu 10.10. Welcome, Maverick Meerkat!

17. By Martin Pitt

[ Peter Eisentraut ]
* Add initial support for 9.0 (pre-release snapshots for now):
  - cleanpg: Remove 9.0 clusters.
  - pg_createcluster: Clone configure_8_4() as configure_9_0().
  - pg_upgradecluster: Migrate obsolete/changed configuration parameters.
* pg_wrapper: Fix spelling.
* In maintainer scripts, instead of "#!/bin/sh -e" use a separate "set -e",
  per lintian.
* debian/control: Bump Standards-Version to 3.8.4; no changes necessary.

[ Martin Pitt ]
* cleanpg: Do not remove the /var/log/postgresql/ directory itself, only its
  contents, to avoid logcheck falling over.
* t/020_create_sql_remove.t: Do not consider $PG_GRANDPARENT_PID an unsafe
  environment, it's set by 9.0 servers.
* t/060_obsolete_confparams.t: Add a full configuration for 8.4, so that
  upgrades to 9.0 can be tested.
* t/052_upgrade_encodings.t: Specify 'C' locale when creating the ASCII
  encoded test database for versions >= 8.4, otherwise we have a
  locale/encoding mismatch.
* t/TestLib.pm: Abort tests on FAILURE=shell and the shell exits with
  nonzero. This avoids endless followup error shells once it's clear that
  the following ones are doomed anyway.
* PgCommon.pm: Add new function get_db_locales() which reads lc_ctype and
  lc_locales for a particular database. This replaces the
  get_cluster_locales() function for >= 8.4 server versions (which moved
  from global to per-database locales).
* pg_upgradecluster: Use get_db_locales() instead of get_cluster_locales()
  for upgrades from >= 8.4.
* pg_ctlcluster: Drop check for valid locales. We can only determine them
  (using pg_controldata through get_cluster_locales()) for <= 8.3 clusters,
  which are obsolete in sid/lenny. For >= 8.4 clusters, we need to start the
  server to find out the locales, at which point the check is pointless.
* pg_upgradecluster: Run pg_restore with --disable-triggers when running in
  --data-only mode. (Closes: #579768)
* t/020_create_sql_remove.t: We now assume that /var/log/postgresql/ always
  exists, and is writable by the "postgres" user. Thus if we delete the log,
  running "pg_ctlcluster start" should be able to recreate it. This fixes
  the 020_create_sql_remove.t test failure in "umask 077" mode, and uncovers
  some more inconsistencies wrt. log directory permissions.
* pg_createcluster, testsuite: Fix permissions of /var/log/postgresql/ if
  this script creates it.

16. By Martin Pitt

* t/001_packages.t: Check for ".utf8" locale instead of ".UTF-8". This
  seems to be the new canonical spelling now.
* t/TestLib.pm, check_clean(): Require that /var/log/postgresql/ still
  exists. If we remove it entirely, current versions of logcheck complain
  loudly. This was already fixed a while ago in squeeze, this will ensure it
  stays so. (Closes: #576180)
* t/040_upgrade.t: Check upgrade of database and table ACL, an ALTER
  DATABASE option, and correct upgrading of a custom pg_hba.conf. This
  reproduces #543506.
* pg_upgradecluster: Rework logic to use "pg_dumpall -s" instead of
  "pg_dumpall -g", to also catch ACLs, global settings, and the like. Drop
  manual database creation and encoding special-cases, since those are being
  taken care of by the -s commands already. (Closes: #543506, #562676)
* pg_upgradecluster: Re-enable connections after the upgrade is done, not in
  between.
* t/120_pg_upgradecluster_scripts.t: Upgrade scripts should not create
  tables in the "init" phase, since they will be overwritten during
  dump/restore. Change the test case to move table changes into the "finish"
  phase, and update documentation in pg_upgradecluster to point out this
  potential problem.
* debian/init.d-functions, status(): More appropriate output formatting for
  "status" init script commands. (Closes: #522679)

15. By Martin Pitt

debian/postgresql-common.postinst: Fix upgrade failure if no tsearch
dictionaries exist. (Closes: #565966)

14. By Martin Pitt

* pg_updatedicts: Process dictionaries from /usr/share/hunspell, too.
* debian/control: Add Vcs-* fields.
* Add standard license headers to all programs, and update copyright.
* debian/copyright: Update to DEP-5 (Machine-readable debian/copyright), and
  drop Oliver's copyright, since there is nothing left that was written by
  him.
* pg_updatedicts: Do not create system_<locale>.{affix,dict} symlinks any
  more, but use the actual names that upstream PostgreSQL looks for (without
  system_ prefix). This allows us to drop the system_ lookup patch from
  PostgreSQL. Update t/150_tsearch_stemming.t accordingly and remove the old
  system_* symlinks on upgrade in debian/postgresql-common.postinst.

13. By Martin Pitt

* Move the lsb-release dependency of p-common to a recommends of
  p-client-common. (Closes: #562642)
* Drop Oliver from Uploaders:, he has not been active any more for years.
* t/005_PgCommon.t: Add '.' to library search path, so that this prefers
  testing PgCommon.pm from the source tree.
* t/005_PgCommon.t: Add test cases for {set,disable,replace}_conf_value().
* PgCommon.pm, replace_conf_value(): Do not append the new parameter if the
  old is not found.
* PgCommon.pm, {set,disable,replace}_conf_value(): Intercept errors on
  writing data (which could happen when running out of disk space). Write
  into a new file and rename it in the end, instead of directly writing into
  the original file. Thanks to Yann Dirson for the original patch.
  (Closes: #549206)
* pg_createcluster: Change effective gid as well when setting the socket
  directory, to avoid moving postgresql.conf's group to root (which would
  happen with the previous change).
* Add t/042_upgrade_tablespaces.t: Test upgrading a cluster with
  tablespaces. Since this is not currently supported (and hard to do in an
  automated fashion due to the nature of tablespaces), just check that this
  errors out with a sane message and does not do any damage.
* pg_upgradecluster: Add an early check for tablespaces. (Closes: #523574)
* pg_lsclusters: Fix undefined value if owneruid cannot be determined (which
  can happen in special setups). Thanks to Daniel Pittman! (Closes: #541434)
* t/020_create_sql_remove.t: Verify permissions of the data and
  configuration directories.
* pg_createcluster: Make /etc/postgresql/<version> and
  /var/lib/postgresql/<version> owned by 'postgres', so that they can be
  administrated with 'postgres' privileges. Update t/030_errors.t
  accordingly. (LP: #236704, Closes: #525294)
* t/020_create_sql_remove.t: Check that a missing log file is recreated by
  pg_ctlcluster (if it has enough permissions).
* pg_ctlcluster: Make re-creation of log file actually work. (LP: #391119,
  #372476)

12. By Martin Pitt

* debian/postgresql-common.postinst: Create /var/log/postgresql/ if it does
  not exist, before trying to chown/chmod it.
* t/001_packages.t: Check that -server-dev-* is installed (for
  t/140_pg_config.t)
* t/001_packages.t: Check that the system default locale is an UTF-8 one.
* debian/supported-versions: Add Ubuntu 10.04.
* t/050_encodings.t: Disable Russian error message encoding test for now,
  since Russian translations were dropped upstream (too incomplete).

Branch metadata

Branch format:
Branch format 7
Repository format:
Bazaar repository format 2a (needs bzr 1.16 or later)
Stacked on:
lp:ubuntu/natty/postgresql-common
This branch contains Public information 
Everyone can see this information.

Subscribers