Re: [sisyphus] [cyber] I: Sisyphus-20091120 packages: +13! -4 +88 (9441)

Re: [sisyphus] [cyber] I: Sisyphus-20091120 packages: +13! -4 +88 (9441)

[Slava Semushin]

21 ноября 2009 г. 2:58 пользователь QA Team Robot <qa@altlinux.org> написал:
[...]
> apt - Debian's Advanced Packaging Tool with RPM support
> * Fri Nov 20 2009 Alexey Tourbin <at@altlinux> 0.5.15lorg2-alt30
> - added support for changelogs in "apt-cache show"
> * Sun Sep 27 2009 Alexey Tourbin <at@altlinux> 0.5.15lorg2-alt29

Хотелось бы узнать как это работает?

[...]

-- 
Slava Semushin

Re: [sisyphus] I: Sisyphus-20091120 packages: +13! -4 +88 (9441)

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 623 bytes --]

On Sat, Nov 21, 2009 at 09:01:39PM +0600, Slava Semushin wrote:
> 21 ноября 2009 г. 2:58 пользователь QA Team Robot <qa@altlinux.org> написал:
> [...]
> > apt - Debian's Advanced Packaging Tool with RPM support
> > * Fri Nov 20 2009 Alexey Tourbin <at@altlinux> 0.5.15lorg2-alt30
> > - added support for changelogs in "apt-cache show"
> > * Sun Sep 27 2009 Alexey Tourbin <at@altlinux> 0.5.15lorg2-alt29
> 
> Хотелось бы узнать как это работает?

"apt-cache show" будет показывать несколько последних записей
в changelog'е пакета.  Когда администратор girar-builder смёржит
мои изменения в работающую систему.

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

Re: [sisyphus] I: Sisyphus-20091120 packages: +13! -4 +88 (9441)

[Slava Semushin]

21 ноября 2009 г. 23:14 пользователь Alexey Tourbin <at@altlinux.ru> написал:
> On Sat, Nov 21, 2009 at 09:01:39PM +0600, Slava Semushin wrote:
>> 21 ноября 2009 г. 2:58 пользователь QA Team Robot <qa@altlinux.org> написал:
>> [...]
>> > apt - Debian's Advanced Packaging Tool with RPM support
>> > * Fri Nov 20 2009 Alexey Tourbin <at@altlinux> 0.5.15lorg2-alt30
>> > - added support for changelogs in "apt-cache show"
>> > * Sun Sep 27 2009 Alexey Tourbin <at@altlinux> 0.5.15lorg2-alt29
>>
>> Хотелось бы узнать как это работает?
>
> "apt-cache show" будет показывать несколько последних записей
> в changelog'е пакета.

"Несколько" это сколько? По какому критерию они показываются? Вызовет
ли это увеличение размера скачиваемых файлов при apt-get update? Если
да, то насколько?

Зачем, кстати, это понадобилось?

> Когда администратор girar-builder смёржит
> мои изменения в работающую систему.

Не очень понял причем тут "администратор girar-builder"? Обновился
пакет apt, попал в Сизиф, скоро он окажется у меня, как у
пользователя. Где связь с "администратор girar-builder"?


-- 
Slava Semushin

Re: [sisyphus] I: Sisyphus-20091120 packages: +13! -4 +88 (9441)

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 2052 bytes --]

On Sat, Nov 21, 2009 at 11:20:02PM +0600, Slava Semushin wrote:
> 21 ноября 2009 г. 23:14 пользователь Alexey Tourbin <at@altlinux.ru> написал:
> > On Sat, Nov 21, 2009 at 09:01:39PM +0600, Slava Semushin wrote:
> >> 21 ноября 2009 г. 2:58 пользователь QA Team Robot <qa@altlinux.org> написал:
> >> [...]
> >> > apt - Debian's Advanced Packaging Tool with RPM support
> >> > * Fri Nov 20 2009 Alexey Tourbin <at@altlinux> 0.5.15lorg2-alt30
> >> > - added support for changelogs in "apt-cache show"
> >> > * Sun Sep 27 2009 Alexey Tourbin <at@altlinux> 0.5.15lorg2-alt29
> >>
> >> Хотелось бы узнать как это работает?
> >
> > "apt-cache show" будет показывать несколько последних записей
> > в changelog'е пакета.
> 
> "Несколько" это сколько? По какому критерию они показываются? Вызовет
> ли это увеличение размера скачиваемых файлов при apt-get update? Если
> да, то насколько?

По критерию --chagelog-since=2009-01-01.

genbasedir:
   --changelog-since=DATE     Save package changelogs; copy changelog entries
                              newer than DATE, and also one preceding entry

Размер x86_64/base/pkglist.classic.bz2 увеличится примерно с 2.8M
до 3.3M.  Скоро увидим более точно, я думаю.  В genpkglist была
реализована одна оптимизация, чтобы заметно сократить размер
pkglist.classic.bz2: пакеты группируются по src.rpm.  То есть
пакеты с одинаковыми changelog'ами идут подряд, за счет чего pkglist
лучше ужимается.

> Зачем, кстати, это понадобилось?

Ну как сказать.  От безделья в носу ковырял, решил чего-нибудь отчебучить.

> > Когда администратор girar-builder смёржит
> > мои изменения в работающую систему.
> 
> Не очень понял причем тут "администратор girar-builder"? Обновился
> пакет apt, попал в Сизиф, скоро он окажется у меня, как у
> пользователя. Где связь с "администратор girar-builder"?

Связь что сизиф должен генрироваться определенным образом: genpkglist
будет включать в репозиторий changelog'и пакетов.  После 'apt-get
update' эти changelog'и станут доступны локально.

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

Re: [sisyphus] I: Sisyphus-20091120 packages: +13! -4 +88 (9441)

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 192 bytes --]

On Sat, Nov 21, 2009 at 08:37:13PM +0300, Alexey Tourbin wrote:
> По критерию --chagelog-since=2009-01-01.
А почему не "$(date '+%Y-%m-%d' --date='year ago')"?

-- 
Regards,
Sir Raorn.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [sisyphus] I: Sisyphus-20091120 packages: +13! -4 +88 (9441)

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 507 bytes --]

On Sat, Nov 21, 2009 at 08:14:20PM +0300, Alexey Tourbin wrote:
> "apt-cache show" будет показывать несколько последних записей
> в changelog'е пакета.
Физически этот текст будет добавляться к Description.  Не тру.
Гораздо веселее было бы если бы libapt-pkg пердоставлял API для
вытаскивания новых записей с версии "Version-Release", тогда в
той же aptitude можно было бы непосредственно перед установкой
посмотреть что в пакете поменялось.  Очень этого не хватает...


-- 
Regards,
Sir Raorn.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [sisyphus] I: Sisyphus-20091120 packages: +13! -4 +88 (9441)

[Sergey Vlasov]

[-- Attachment #1: Type: text/plain, Size: 1448 bytes --]

On Sat, Nov 21, 2009 at 08:51:15PM +0300, Alexey I. Froloff wrote:
> On Sat, Nov 21, 2009 at 08:14:20PM +0300, Alexey Tourbin wrote:
> > "apt-cache show" будет показывать несколько последних записей
> > в changelog'е пакета.
> Физически этот текст будет добавляться к Description.

На самом деле это отдельное поле Changelog.

>  Не тру.

+1, но по другой причине - выбор этих "нескольких последних записей"
в текущем варианте малопригоден для использования.  В частности, для
пакета, установленного локально, но отсутствующего в репозиториях
(что-то местной сборки, или выкинутое из Сизифа, но оставшееся в
системе), выдаются все записи changelog, имеющиеся в пакете - такая
простыня никому не нужна, а отключить её хотя бы через apt.conf
нельзя.

> Гораздо веселее было бы если бы libapt-pkg пердоставлял API для
> вытаскивания новых записей с версии "Version-Release", тогда в
> той же aptitude можно было бы непосредственно перед установкой
> посмотреть что в пакете поменялось.  Очень этого не хватает...

+1.

Вообще хотелось бы иметь, например, команду для apt-shell, выводящую
записи changelog для пакетов, выбранных для обновления, с учётом
текущих установленных версий пакетов - чтобы выводились только
изменения после установленной сборки (как в письмах от QA Robot).
Ну и/или опцию для apt-get, чтобы такая информация выдавалась перед
началом установки пакетов (такое есть в yum: yum update --changelog).

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [sisyphus] I: Sisyphus-20091120 packages: +13! -4 +88 (9441)

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 560 bytes --]

On Sat, Nov 21, 2009 at 09:30:30PM +0300, Sergey Vlasov wrote:
> > > "apt-cache show" будет показывать несколько последних записей
> > > в changelog'е пакета.
> > Физически этот текст будет добавляться к Description.
> На самом деле это отдельное поле Changelog.
Даже не поле, просто блок текста, который есть только в выводе
RecordParser::GetRec() - The record in raw text, in std Debian
format.  Никаких других функций для получения этого ченжлога
RecordParser не предоставляет (та же aptitude не использует
GetRec()).

-- 
Regards,
Sir Raorn.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

[sisyphus] [PATCH] Access to changelogs via record parser interface

[Alexey I. Froloff]

Add RecordParser::Changelog() and SrcRecordParser::Changelog() functions
that returns raw changelog text.

libapt-pkg SONAME changed according to libtool versioning rules.

Signed-off-by: Alexey I. Froloff <raorn@altlinux.org>
---
 apt/apt-pkg/Makefile.am          |    2 +-
 apt/apt-pkg/deb/debrecords.cc    |    8 ++++++++
 apt/apt-pkg/deb/debrecords.h     |    1 +
 apt/apt-pkg/deb/debsrcrecords.h  |    1 +
 apt/apt-pkg/pkgrecords.h         |    1 +
 apt/apt-pkg/rpm/rpmrecords.cc    |   14 ++++++++++++++
 apt/apt-pkg/rpm/rpmrecords.h     |    1 +
 apt/apt-pkg/rpm/rpmsrcrecords.cc |   13 +++++++++++++
 apt/apt-pkg/rpm/rpmsrcrecords.h  |    1 +
 apt/apt-pkg/srcrecords.h         |    1 +
 10 files changed, 42 insertions(+), 1 deletions(-)

diff --git a/apt/apt-pkg/Makefile.am b/apt/apt-pkg/Makefile.am
index ebf0398..ca563d7 100644
--- a/apt/apt-pkg/Makefile.am
+++ b/apt/apt-pkg/Makefile.am
@@ -2,7 +2,7 @@
 lib_LTLIBRARIES = libapt-pkg.la
 
 libapt_pkg_la_LIBADD = @RPMLIBS@
-libapt_pkg_la_LDFLAGS = -version-info 2:0:0 -release @GLIBC_VER@-@LIBSTDCPP_VER@
+libapt_pkg_la_LDFLAGS = -version-info 3:0:1 -release @GLIBC_VER@-@LIBSTDCPP_VER@
 
 AM_CPPFLAGS = -DLIBDIR=\"$(libdir)\"
 
diff --git a/apt/apt-pkg/deb/debrecords.cc b/apt/apt-pkg/deb/debrecords.cc
index aa6253a..7b31626 100644
--- a/apt/apt-pkg/deb/debrecords.cc
+++ b/apt/apt-pkg/deb/debrecords.cc
@@ -92,6 +92,14 @@ string debRecordParser::LongDesc()
    return Section.FindS("Description");
 }
 									/*}}}*/
+// RecordParser::Changelog - Return package changelog if any		/*{{{*/
+// ---------------------------------------------------------------------
+/* */
+string debRecordParser::Changelog()
+{
+   return string("");
+}
+									/*}}}*/
 // RecordParser::SourcePkg - Return the source package name if any	/*{{{*/
 // ---------------------------------------------------------------------
 /* */
diff --git a/apt/apt-pkg/deb/debrecords.h b/apt/apt-pkg/deb/debrecords.h
index 8760c38..a71e9bd 100644
--- a/apt/apt-pkg/deb/debrecords.h
+++ b/apt/apt-pkg/deb/debrecords.h
@@ -44,6 +44,7 @@ class debRecordParser : public pkgRecords::Parser
    virtual string ShortDesc();
    virtual string LongDesc();
    virtual string Name();
+   virtual string Changelog();
 
    virtual void GetRec(const char *&Start,const char *&Stop);
    
diff --git a/apt/apt-pkg/deb/debsrcrecords.h b/apt/apt-pkg/deb/debsrcrecords.h
index ff14045..18a9339 100644
--- a/apt/apt-pkg/deb/debsrcrecords.h
+++ b/apt/apt-pkg/deb/debsrcrecords.h
@@ -38,6 +38,7 @@ class debSrcRecordParser : public pkgSrcRecords::Parser
    virtual string Version() const {return Sect.FindS("Version");};
    virtual string Maintainer() const {return Sect.FindS("Maintainer");};
    virtual string Section() const {return Sect.FindS("Section");};
+   virtual string Changelog() const {return string("");};
    virtual const char **Binaries();
    virtual bool BuildDepends(vector<BuildDepRec> &BuildDeps, bool ArchOnly);
    virtual unsigned long Offset() {return iOffset;};
diff --git a/apt/apt-pkg/pkgrecords.h b/apt/apt-pkg/pkgrecords.h
index 1bf1f9e..9f831a3 100644
--- a/apt/apt-pkg/pkgrecords.h
+++ b/apt/apt-pkg/pkgrecords.h
@@ -67,6 +67,7 @@ class pkgRecords::Parser
    virtual string ShortDesc() {return string();};
    virtual string LongDesc() {return string();};
    virtual string Name() {return string();};
+   virtual string Changelog() {return string();};
    
    // The record in binary form
    virtual void GetRec(const char *&Start,const char *&Stop) {Start = Stop = 0;};
diff --git a/apt/apt-pkg/rpm/rpmrecords.cc b/apt/apt-pkg/rpm/rpmrecords.cc
index eb64fe9..e3f6852 100644
--- a/apt/apt-pkg/rpm/rpmrecords.cc
+++ b/apt/apt-pkg/rpm/rpmrecords.cc
@@ -171,6 +171,20 @@ string rpmRecordParser::LongDesc()
    return Ret;
 }
 									/*}}}*/
+// RecordParser::Changelog - Return package changelog if any		/*{{{*/
+// -----------------------------------------------
+string rpmRecordParser::Changelog()
+{
+   char *str;
+
+   str = headerSprintf(HeaderP,
+         "[* %{CHANGELOGTIME:day} %{CHANGELOGNAME}\n%{CHANGELOGTEXT}\n\n]",
+         rpmTagTable, rpmHeaderFormats, NULL);
+
+   return string(str?str:"");
+}
+
+									/*}}}*/
 // RecordParser::SourcePkg - Return the source package name if any	/*{{{*/
 // ---------------------------------------------------------------------
 /* */
diff --git a/apt/apt-pkg/rpm/rpmrecords.h b/apt/apt-pkg/rpm/rpmrecords.h
index 71e3325..d8a72d5 100644
--- a/apt/apt-pkg/rpm/rpmrecords.h
+++ b/apt/apt-pkg/rpm/rpmrecords.h
@@ -59,6 +59,7 @@ class rpmRecordParser : public pkgRecords::Parser
    virtual string ShortDesc();
    virtual string LongDesc();
    virtual string Name();
+   virtual string Changelog();
    
    inline Header GetRecord() { return HeaderP; };
 
diff --git a/apt/apt-pkg/rpm/rpmsrcrecords.cc b/apt/apt-pkg/rpm/rpmsrcrecords.cc
index 43b25fa..3429309 100644
--- a/apt/apt-pkg/rpm/rpmsrcrecords.cc
+++ b/apt/apt-pkg/rpm/rpmsrcrecords.cc
@@ -191,6 +191,19 @@ string rpmSrcRecordParser::Section() const
    return string(rc?str:"");
 }
 
+// SrcRecordParser::Changelog - Package changelog
+// ----------------------------------------------
+string rpmSrcRecordParser::Changelog() const
+{
+   char *str;
+
+   str = headerSprintf(HeaderP,
+         "[* %{CHANGELOGTIME:day} %{CHANGELOGNAME}\n%{CHANGELOGTEXT}\n\n]",
+         rpmTagTable, rpmHeaderFormats, NULL);
+
+   return string(str?str:"");
+}
+
 unsigned long rpmSrcRecordParser::Offset() 
 {
     return Handler->Offset();
diff --git a/apt/apt-pkg/rpm/rpmsrcrecords.h b/apt/apt-pkg/rpm/rpmsrcrecords.h
index 6c96a40..41505ae 100644
--- a/apt/apt-pkg/rpm/rpmsrcrecords.h
+++ b/apt/apt-pkg/rpm/rpmsrcrecords.h
@@ -48,6 +48,7 @@ public:
    virtual string Version() const;
    virtual string Maintainer() const;
    virtual string Section() const;
+   virtual string Changelog() const;
    virtual const char **Binaries();
    virtual unsigned long Offset();
    virtual string AsStr();
diff --git a/apt/apt-pkg/srcrecords.h b/apt/apt-pkg/srcrecords.h
index 006c9d3..8bc013f 100644
--- a/apt/apt-pkg/srcrecords.h
+++ b/apt/apt-pkg/srcrecords.h
@@ -70,6 +70,7 @@ class pkgSrcRecords
       virtual string Version() const = 0;
       virtual string Maintainer() const = 0;
       virtual string Section() const = 0;
+      virtual string Changelog() const = 0;
       virtual const char **Binaries() = 0;   // Ownership does not transfer
 
       virtual bool BuildDepends(vector<BuildDepRec> &BuildDeps, bool ArchOnly) = 0;
-- 
1.6.5.3

Re: [sisyphus] [PATCH] Access to changelogs via record parser interface

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 435 bytes --]

On Sun, Nov 22, 2009 at 04:58:25AM +0300, Alexey I. Froloff wrote:
> +   str = headerSprintf(HeaderP,
> +         "[* %{CHANGELOGTIME:day} %{CHANGELOGNAME}\n%{CHANGELOGTEXT}\n\n]",
> +         rpmTagTable, rpmHeaderFormats, NULL);
> +
> +   return string(str?str:"");
Говорят, конструктор немного копирует строку из const char *,
поэтому str было бы неплохо освободить...

P.S. Ненавижу си+плюс.

-- 
Regards,
Sir Raorn.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [sisyphus] [PATCH] Access to changelogs via record parser interface

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 420 bytes --]

On Sun, Nov 22, 2009 at 02:58:48PM +0300, Alexey I. Froloff wrote:
> Говорят, конструктор немного копирует строку из const char *,
> поэтому str было бы неплохо освободить...
Я закоммитил себе в гит это изменение.  Пример использования в
aptitude можно посмотреть здесь -
http://imagebin.ca/view/SpKv6C.html

Принимаются предложения по раскрашиванию строки
date-maintainer-version.

-- 
Regards,
Sir Raorn.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [sisyphus] [PATCH] Access to changelogs via record parser interface

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 430 bytes --]

On Sun, Nov 22, 2009 at 07:16:44PM +0300, Alexey I. Froloff wrote:
> On Sun, Nov 22, 2009 at 02:58:48PM +0300, Alexey I. Froloff wrote:
> > Говорят, конструктор немного копирует строку из const char *,
> > поэтому str было бы неплохо освободить...
> Я закоммитил себе в гит это изменение.  Пример использования в
> aptitude можно посмотреть здесь -
> http://imagebin.ca/view/SpKv6C.html
ping?

-- 
Regards,
Sir Raorn.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [sisyphus] [PATCH] Access to changelogs via record parser interface

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 529 bytes --]

On Thu, Nov 26, 2009 at 03:17:22AM +0300, Alexey I. Froloff wrote:
> On Sun, Nov 22, 2009 at 07:16:44PM +0300, Alexey I. Froloff wrote:
> > On Sun, Nov 22, 2009 at 02:58:48PM +0300, Alexey I. Froloff wrote:
> > > Говорят, конструктор немного копирует строку из const char *,
> > > поэтому str было бы неплохо освободить...
> > Я закоммитил себе в гит это изменение.  Пример использования в
> > aptitude можно посмотреть здесь -
> > http://imagebin.ca/view/SpKv6C.html
> ping?

later

> -- 
> Regards,
> Sir Raorn.

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

[sisyphus] shallow changelogs (was: I: Sisyphus-20091120 packages: +13! -4 +88 (9441))

[Michael Shigorin]

On Sat, Nov 21, 2009 at 08:51:15PM +0300, Alexey I. Froloff wrote:
> > "apt-cache show" будет показывать несколько последних записей
> > в changelog'е пакета.
> Физически этот текст будет добавляться к Description.  Не тру.
> Гораздо веселее было бы если бы libapt-pkg пердоставлял API для
> вытаскивания новых записей с версии "Version-Release", тогда в
> той же aptitude можно было бы непосредственно перед установкой
> посмотреть что в пакете поменялось.  Очень этого не хватает...

Не то слово, и на сайте бы тоже вывешивать.  Удобство отсмотра
таких ченжлогов оценил по TYPO3 Extension Manager, когда сделали.

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/