Your message dated Tue, 14 Jun 2011 15:22:42 +0200
with message-id <[email protected]>
and subject line Re: Bug#621472: buildd.debian.org: version field in table
pkg_history should be of type debversion
has caused the Debian Bug report #621472,
regarding buildd.debian.org: version field in table pkg_history should be of
type debversion
to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)
--
621472: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=621472
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: buildd.debian.org
Severity: normal
Hi.
The pkg_history table is defined as follows:
CREATE TABLE pkg_history (
package character varying NOT NULL,
distribution character varying NOT NULL,
version character varying NOT NULL,
"timestamp" timestamp without time zone NOT NULL,
result character varying NOT NULL,
build_time integer,
disk_space bigint
);
version here is of type "character varying", but maybe it should use
"debversion" instead? (to take advantage of debversion_cmp).
The effect appears on [1] where "1:9.7.3.dfsg-1~squeeze1~bpo50+1"
comes _before_ "1:9.7.3.dfsg-1".
[1] https://buildd.debian.org/status/logs.php?pkg=bind9
(Same remark goes for "packages" and "transactions" tables).
Regards,
-- System Information:
Debian Release: 6.0.1
APT prefers stable
APT policy: (990, 'stable'), (500, 'stable-updates'), (500,
'proposed-updates')
Architecture: amd64 (x86_64)
Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.utf8, LC_CTYPE=en_US.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
--- End Message ---
--- Begin Message ---
On Thu, Apr 07, 2011 at 11:50:59AM +0200, Mehdi Dogguy wrote:
> On 04/07/2011 10:03 AM, Mehdi Dogguy wrote:
> > version here is of type "character varying", but maybe it should use
> > "debversion" instead? (to take advantage of debversion_cmp).
> I casted version as debversion, and it seems to work fine.
> Still, maybe it should be the default.
As I said on IRC it's not easy. Anyway, with the big migration
yesterday I also fixed this. You still need to cast it because you
use the output of query_source_package but that query should be
rewritten using packages_public anyway.
Kind regards
Philipp Kern
--- End Message ---
