Hi,

Extensive discussion between the bug reporter and the Ethereal development
team lead to the conclusion that the patch is incorrect and should be
dropped.

Thanx,
Jaap

---------- Forwarded message ----------
Date: Tue, 25 Apr 2006 12:57:18 +0200
From: Joost Yervante Damad <[EMAIL PROTECTED]>
To: Jaap Keuter <[EMAIL PROTECTED]>
Cc: Anders Broman <[EMAIL PROTECTED]>
Subject: Re: SV: Debian Bug report logs - #316082

Hi Jaap, Anders,

this is my suggestion:

go with the current implementation. If the need arises for me to use diameter
again, I will surely use ethereal again for it, and if it's still not working
I'll do a proper investigate.

Sorry I could not be of more help.

Congrats with the 1.0!

Greetings, Joost


On Tuesday 25 April 2006 07:57, Jaap Keuter wrote:
> Hi Joost,
>
> We're steadily approaching Ethereal 1.0 (yes, it's freezing in hell :)
> so we should get to a conclusion on this before Frederic starts packaging
> that release.
>
> Thanx,
> Jaap
>
> On Sun, 2 Apr 2006, Joost Yervante Damad wrote:
> > Hi Jaap, Anders,
> >
> > sorry, I've been occupied by other things and completely forgot about it
> > as the work I needed diameter support for is long finished :)
> >
> > I'll try and find some time for this this week.
> >
> > Joost
> >
> > On Monday 20 March 2006 20:30, Anders Broman wrote:
> > > Hi,
> > > The "fix" is not correct. Actually it breaks the loading of xml files
> > > if I remember correctly.
> > > Joost said he'd test his xml files further as he found some problems
> > > with them after I showed him how to get better debug info from the
> > > diameter dissector. I have also subsequently updated our own files. If
> > > there still is a problem it needs a different solution.
> > > Brg
> > > Anders
> > >
> > > -----Ursprungligt meddelande-----
> > > Fr?n: Jaap Keuter [mailto:[EMAIL PROTECTED]
> > > Skickat: den 20 mars 2006 16:29
> > > Till: Joost Yervante Damad
> > > Kopia: Anders Broman
> > > ?mne: Debian Bug report logs - #316082
> > >
> > > Hi Joost,
> > >
> > > ethereal: fails to correctly add diameter vendors from the dictionary
> > > xml files
> > >
> > > You've discussed this issue with Anders Broman and worked out the way
> > > to go forward. Still the Ethereal code base is the same and the bug
> > > report is still open. What was the conclusion of your discussion?
> > > Should the code be changed or the bugreport, including patch, be
> > > dropped? We have to come to some conclusion here in order to
> > > synchronize Debian and 'Rest Of World' use of diameter files.
> > >
> > > Thanx,
> > > Jaap

-- 
The planet Andete is famous for it's killer edible poets.


Reply via email to