reopen 21659
severity 21659 wishlist
tags 21659 patch
thanks

Hi,

sorry for the delay

Bernd wrote:
> On Fri, Sep 15, 2006 at 12:48:51PM -0700, Debian Bug Tracking System 
wrote:
>> dselect only shows "installed control info" for obsolete packages,
>> and given that this report has been ignored for 8+ years it is safe 
>> to say that is the intended behaviour.
>
> It does not show that info by default, and it does not show it in the
> description column. 
>
> If i am on a local/obsolote package it prints "package - no
> description available". Which it is clearly wrong, since pressing "i" 
> will show me the description. So it should be "for your inconvinience 
> we do not show the description you would expect here until you press 
> <i>". Fine with me :)    

:-)

> Sorry I dont understand the technical issues which arise from showing 
> that info, I am just a user who would expect that info and find it 
> helpfull (if I would actually use dselect).  

dselect is coded to only show descriptions for *available* packages, 
that is the "intended behaviour" I mentioned and what appears unlikely 
to change given the length of time the report has languished in the 
BTS.

I could only guess at why it was decided that descriptions should only 
be shown for Available packages, so I'll refrain from doing so, but it 
does serve to draw attention to the Obsolete ones (which is generally a 
good thing, imo.)

Technically, from pkginfo.cc:

void packagelist::itd_description() {
  if (table[cursorline]->pkg->name) {
    whatinfovb(_("description of "));
    whatinfovb(table[cursorline]->pkg->name);

    const char *m= table[cursorline]->pkg->available.description;
    if (!m || !*m) m= _("no description available.");
    const char *p= strchr(m,'\n');
    ...

Getting dselect to always show a Description would require an additional 
test and attempt to set "m", afaict, something like:

    const char *m= table[cursorline]->pkg->available.description;
    if (!m || !*m) m= table[cursorline]->pkg->installed.description;
    if (!m || !*m) m= _("no description available.");

Which doesn't seem too difficult, unreasonable, or likely to introduce 
bugs... reopening as a wishlist item and tagging with "patch", just 
because I may have been a bit overzealous with this one.


- Bruce


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to