Re: mpg321 at mentors

2012-09-17 Thread Nanakos Chrysostomos

Dear Bart,



On 17 Σεπ 2012, at 7:37, Bart Martens ba...@debian.org wrote:


On Sun, Sep 16, 2012 at 10:06:51PM +0300, Nanakos Chrysostomos wrote:

Hi Bart,

On Wed, Sep 12, 2012 at 07:08:18PM +, Bart Martens wrote:

Hi Nanakos,

I had a look at mpg321 at mentors uploaded there on 2012-08-27  
09:13.


It is not clear to me on bug 685679 what you mean with does not  
perform very
well and not very stable.  I don't see how this makes mpg321  
unusable and
how it would break other applications.  Please explain on the  
bug report
what's wrong and why you believe that disabling the -b option  
(actually making
mpg321 silently ignore the -b option) would be the best approach  
in this

context.


I hope my last message covers your questions.


I currently see on bug 685679 only the initial message of 23 Aug 2012.


Attached the message you asked on bug 685679. Does this cover you?
Does the package need any improvements?

Regards,
Chris.






--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/68454122-95f1-44a8-b92d-8c3ef7330...@wired-net.gr



Re: mpg321 at mentors

2012-09-17 Thread Bart Martens
Hello Nanakos,

On Tue, Sep 18, 2012 at 12:05:41AM +0300, Nanakos Chrysostomos wrote:
 On 17 Σεπ 2012, at 7:37, Bart Martens ba...@debian.org wrote:
 On Sun, Sep 16, 2012 at 10:06:51PM +0300, Nanakos Chrysostomos wrote:
 I hope my last message covers your questions.
 I currently see on bug 685679 only the initial message of 23 Aug 2012.
 Attached the message you asked on bug 685679. Does this cover you?
 Does the package need any improvements?

You added good info to the bug report.  I added some more thoughts on the bug
report now.  Can you have a look at them and (if you agree with them) update
the package at mentors accordingly ?

Regards,

Bart Martens


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20120918051020.gc22...@master.debian.org



Re: mpg321 at mentors

2012-09-16 Thread Nanakos Chrysostomos
Hi Bart,

On Wed, Sep 12, 2012 at 07:08:18PM +, Bart Martens wrote:
 Hi Nanakos,

 I had a look at mpg321 at mentors uploaded there on 2012-08-27 09:13.

 It is not clear to me on bug 685679 what you mean with does not perform very
 well and not very stable.  I don't see how this makes mpg321 unusable and
 how it would break other applications.  Please explain on the bug report
 what's wrong and why you believe that disabling the -b option (actually making
 mpg321 silently ignore the -b option) would be the best approach in this
 context.

I hope my last message covers your questions.

 If the -b option does nothing then the manpage could be modified to mention
 that.

Fixed.


 The entries 0.3.2-2 and 0.3.2-1.1 in debian/changelog have lines with one
 leading space too many.


Fixed

 The uncommenting of dh_testroot in debian/rules is not mentioned in
 debian/changelog.


My mistake, I haven't copied the latest debian/rules file from the last upload
of Emfox Zhou. Now fixed.

It would be great if you could sponsor the package. My next step is to 
communicate
with the Release Managers to unblock the package.

Regards,
Chris.



signature.asc
Description: Digital signature


Re: mpg321 at mentors

2012-09-16 Thread Bart Martens
On Sun, Sep 16, 2012 at 10:06:51PM +0300, Nanakos Chrysostomos wrote:
 Hi Bart,
 
 On Wed, Sep 12, 2012 at 07:08:18PM +, Bart Martens wrote:
  Hi Nanakos,
 
  I had a look at mpg321 at mentors uploaded there on 2012-08-27 09:13.
 
  It is not clear to me on bug 685679 what you mean with does not perform 
  very
  well and not very stable.  I don't see how this makes mpg321 unusable 
  and
  how it would break other applications.  Please explain on the bug report
  what's wrong and why you believe that disabling the -b option (actually 
  making
  mpg321 silently ignore the -b option) would be the best approach in this
  context.
 
 I hope my last message covers your questions.

I currently see on bug 685679 only the initial message of 23 Aug 2012.

Regards,

Bart Martens


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20120917043737.ga31...@master.debian.org



Re: mpg321 at mentors

2012-09-13 Thread Nanakos Chrysostomos
Dear Bart,

On Wed, 12 Sep 2012 19:08:18 +, Bart Martens ba...@debian.org wrote:
 Hi Nanakos,
 
 I had a look at mpg321 at mentors uploaded there on 2012-08-27 09:13.
 
 It is not clear to me on bug 685679 what you mean with does not perform
 very
 well and not very stable.  I don't see how this makes mpg321
unusable
 and
 how it would break other applications.  Please explain on the bug
report
 what's wrong and why you believe that disabling the -b option (actually
 making
 mpg321 silently ignore the -b option) would be the best approach in this
 context.
 

I don't know if you have noticed an abnormal behavior when trying to use
the -b option but occasionally mpg321 crashes, stops playing the song,
plays the song
without syncing with the second process that outputs the song or the
reverse. Never starts
playing an mp3 file or crashes when someone tries to play a file that is
not an mp3. All those situations
does not happen when you are not using -b option. I am trying to solve the
problem but I haven't figured
where the fault is. So I prefer to silently as you say ignore the -b
option and preserve the package
in the previous state, so to provide a stable version that won't make
problems to users or other packages
that try or might use this new feature. 

 If the -b option does nothing then the manpage could be modified to
mention
 that.
 

I will fix that and re-upload the package.

 The entries 0.3.2-2 and 0.3.2-1.1 in debian/changelog have lines with
one
 leading space too many.

Haven't noticed that, I will fix it.

 
 The uncommenting of dh_testroot in debian/rules is not mentioned in
 debian/changelog.

I will fix that also.

Regards,
Chris.

 
 Regards,
 
 Bart Martens


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/369ee4e87b4356657fc880e6bed72...@wired-net.gr