[2010-07-22 16:25] Gerfried Fuchs <rho...@deb.at>
>       Hi!
Hoi

>  First of all, sorry for the late response.

A response, a response, .... great! :-)

> I just recently got adopted
> into the upstream team and am looking through patches now that make
> sense to incorporate there directly instead of having them potential rot
> within Debian alone. ;)

Thanks for working on abook.

I don't use abook's export function anymore because I switched from
mutt to nmh and don't use an alias database there. But anyway the
issue surely needs a fix.


> * markus schnalke <mei...@marmaro.de> [2008-01-09 17:37:11 CET]:
> > Package: abook
> > Version: 0.5.6-3
> > Severity: wishlist
> > 
> > The following command
> > 
> > abook --convert --infile ~/.abook/addressbook --informat abook
> >     --outfile mutt.aliases_test --outformat mutt
> > 
> > does only export the first (primary) email address of each contact.
> > All further (secondary) email addresses are simply ignored.
> > If you want to generate mutt.aliases from abook, you have to create
> > multiple entries for one person in abook - for every email address
> > one.
> > 
> > Also, contacts without email addresses at all, are exported. This
> > leads to entries in the aliases file that are useless.
> 
>  I agree that these are both an annoyance and am taking a look at your
> patch now and trying to bring it up to the 0.6.0 pre2 codebase, quite
> some things have changed, so it isn't a straight forward patch.
> 
>  I got a working version so far, though somehow I got for the rotated
> email as realname all the email addresses, have to check again wether I
> have adjusted something wrongly here, or wether there is some logic
> error involved.
> 
>  Also, the exit you did for no email did make the function quit on the
> first addressbook entry that didn't had an email address attached to it
> - I somehow guess this wasn't your attention. ;)

Surely not. Two and a half year ago, my C knowledge wasn't as good as
it is now. ;-)

> Fixed that in my local
> copy already (moved the "if (*email) {" level to the outermost part of
> the loop).
> 
>  Again, thanks for your suggestion, sorry for the delay, and a fix for
> it will be soon get applied upstream.

Thank you very much for this information mail; that's a great service.

I hope fixing it completely won't be too difficult. Unfortunately, I
can't offer my help, because I'm very busy at the moment. I would have
loved to help, but it's not possible currently.


Have good times.


meillo



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to