Bug#827041: geoip-database-contrib conflicts on install

[Andreas Beckmann]

Control: tag -1 moreinfo

On Sat, 11 Jun 2016 15:07:04 +0200 Nikolas Grottendieck
<in...@nikolasgrottendieck.com> wrote:
> according to bug report 767559 the conflict arising when trying to install geoip-database-contrib should be fixed with version 1.17+nmu1 of geoip-database-contrib.
>
> However, I am currently still getting the error:
>
> The following packages have unmet dependencies:
> geoip-database-contrib : Conflicts: geoid-database but 20150317-1 is installed.
> Conflicts: geoip-database-extra but 20150317-1 is installed.
> The following actions will resolve these dependencies:
>
> Remove the following packages:
> 1) geoid-database
> 2) geoip-database-extra

I just tested this in a minimal jessie chroot with geoip-database
geoip-database-extra installed:

# apt-get install geoip-database-contrib
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following extra packages will be installed:
libicu52 libpsl0 ucf wget
Suggested packages:
cron-daemon
The following packages will be REMOVED:
geoip-database geoip-database-extra
The following NEW packages will be installed:
geoip-database-contrib libicu52 libpsl0 ucf wget
0 upgraded, 5 newly installed, 2 to remove and 0 not upgraded.
Need to get 11.8 kB/7403 kB of archives.
After this operation, 204 kB of additional disk space will be used.
Do you want to continue? [Y/n]


Which looks like the correct solution. AIUI geoip-database-contrib
provides a superset of the functionality from geoip-database and
geoip-database-extra. Therefore removing geoip-database and
geoip-database-extra upon installation of geoip-database-contrib is
correct, since there cannot be two packages providing the same files at
the same time. #767559 was about a missing declaration of this conflict,
leading to a file overwrite error.


Andreas

[Nikolas Grottendieck]

It appears you are correct and I just didn’t understand/read the package descriptions correctly … my bad. To prevent further misunderstandings would it be possible to update the geoip-database-contrib package description and improve the wording?
While it is mentioned on the geoip-database and -extra package description pages that -contrib provides these features there is no mention on the -contrib page explaining this properly. In my opinion it should be more clearly stated that -contrib not only keeps the databases up to date (maybe include the update interval as well?) but also provides all databases the other two packages would provide and can, hence, be used as a substitute.
Given that there’s also a stack overflow question (see http://stackoverflow.com/q/37080475/2920585 ) asking for the differences as well, there are clearly more people who are not too sure about this.

Best regards and thanks for the help,
Nikolas

[Patrick Matthäi]

Am 14.06.2016 um 12:46 schrieb Andreas Beckmann:
> Control: tag -1 - moreinfo
> Control: severity -1 important
> Control: retitle -1 geoip-database-contrib: clarify description

>
> On 2016-06-14 12:25, Nikolas Grottendieck wrote:
>> It appears you are correct and I just didn’t understand/read the package descriptions correctly … my bad.
>> To prevent further misunderstandings would it be possible to update the geoip-database-contrib package description and improve the wording?

> ...
>
> That certainly should be done. Retitling the bug accordingly and
> lowering the severity.
>
>
> Andreas
>

Hi,

for myself the description is good enough: "This package contains a
script that downloads the latest version of the binary"
I do not have got a better idea to re-word the long description (maybe a
native speaker could help here better). Maybe someone has got a patch
for me with an improved description?

--
/*
Mit freundlichem Gruß / With kind regards,
Patrick Matthäi
GNU/Linux Debian Developer

Blog: http://www.linux-dev.org/
E-Mail: pmat...@debian.org
pat...@linux-dev.org
*/

[Nikolas Grottendieck]

On 17.06.2016, at 10:25, Patrick Matthäi <pmat...@debian.org> wrote:
>
> Am 14.06.2016 um 12:46 schrieb Andreas Beckmann:
>> On 2016-06-14 12:25, Nikolas Grottendieck wrote:
>>> It appears you are correct and I just didn’t understand/read the package descriptions correctly … my bad.
>>> To prevent further misunderstandings would it be possible to update the geoip-database-contrib package description and improve the wording?
>> ...
>>
>> That certainly should be done. Retitling the bug accordingly and
>> lowering the severity.
>>
>>
>> Andreas
>>
>
> Hi,
>
> for myself the description is good enough: "This package contains a
> script that downloads the latest version of the binary"
> I do not have got a better idea to re-word the long description (maybe a
> native speaker could help here better). Maybe someone has got a patch
> for me with an improved description?
>

> Patrick Matthäi
>

Hi,

yes, that part of the description most certainly is good enough. However, my (and apparently other’s) confusion stems from the lack of a clear indication that this package can be used as a substitute for geoip-database and geoip-database-extra as opposed to a complementary package.
I think it would suffice if the -extra package description had a sentence along the lines of “This package can be used as a replacement for the packages geoip-database and geoip-database-extra. Additionally, this package contains a script that regularly downloads the latest version of the binary […]” in addition to the current contents of the description currently available.

Best regards,
Nikolas