[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#218995: Patch for #218995



>>Updated patch attached.
>
> Ummmm - where? :-)

Huh. I created a diff in my /tmp, I thought this would be enough. Silly me …
Attached now I hope, but the diff-diff is really not that big.


On Wed, Sep 26, 2012 at 4:51 PM, Daniel Hartwig <mandyke@gmail.com> wrote:
> On 26 September 2012 22:49, Daniel Hartwig <mandyke@gmail.com> wrote:
>> On 26 September 2012 21:50, David Kalnischkies
>> <kalnischkies+debian@gmail.com> wrote:
>>> Combining the two threads suggests that this is a perfect opportunity to
>>> change behavior here by dropping APT-style for Jessie before providing
>>> officially a command which will not allow us to do that in the future.
>>> And as I can't time-travel we will fake it (for apt-cache depends) until
>>> we make it (for all in DepIterator::CompType() ).
>>
>> [Erases plan to push for utf8 output from CompType (natural operators,
>> like “≤”) where supported by the locale.]
>
> … because it seemed to me that the distinction is that CompType is a
> natural representation, and CompTypeDeb is for machine parsing. ?

Kinda, but I think the not-availability of CompTypeDeb from DepIterator
makes the behavior of CompType unexpected. I might be biased here,
but I made this mistake myself at least once while working on EDSP.

"Suddenly" returning UTF-8 characters in a char* will be even more
unexpected. So while I think a pretty-print function might be nice,
it shouldn't be (the default mode of) CompType().

Beside that I am not sure if the operator ≤ is better than <= for users
in a console application like apt-get -- but maybe in T- and GUIs.
Anyway, I think this is the topic of a different discussion after wheezy.


Best regards

David Kalnischkies

Attachment: apt-218995-apt-cache-depends-show-version-v2.diff
Description: Binary data


Reply to: