Bug#498693: [60136ec] Fix for Bug#498693 committed to git

2010-09-26 Thread Giuseppe Iuculano
On 09/24/2010 02:36 PM, Luís Picciochi Oliveira wrote:
 - On my diff, I moved the /usr/share/amule/skins directory from the
 amule package to amule-common.
 You seem to have copied it instead, so now it is provided by both
 packages, although amule depends on amule-common (and this directory
 will come with it).
 
 
 - There are two typos on the new amule-common description:
 
 localization files *and* webserver templates *and* GUI skins, while my
 suggestion only had one and on this sentence.
 
 You probably don't want to install this package alone, but amule or
 amule-daemon instead., while my suggestion was:
 You probably don't want to install this package alone, but amule,
 amule-daemon or amule-utils-gui instead.
 I think it makes sense to suggest amule-utils-gui there, as it can be
 installed independently of any other of these alternatives.
 

Thanks, I commited the fixes:

http://git.debian.org/?p=pkg-amule/amule.git;a=commit;h=c2f0bcfc194f7495e0df84b3c37a7ee6512619fc


http://git.debian.org/?p=pkg-amule/amule.git;a=commit;h=6af73ccb20d3524290a02309de4fd6939bbaca70


Cheers,
Giuseppe.



signature.asc
Description: OpenPGP digital signature


Bug#498693: [60136ec] Fix for Bug#498693 committed to git

2010-09-24 Thread Luís Picciochi Oliveira
Hello,
I might be wrong here, but if I'm seeing this well, your commit didn't quite
match what I suggested.

- On my diff, I moved the /usr/share/amule/skins directory from the amule
package to amule-common.
You seem to have copied it instead, so now it is provided by both packages,
although amule depends on amule-common (and this directory will come with
it).


- There are two typos on the new amule-common description:

localization files *and* webserver templates *and* GUI skins, while my
suggestion only had one and on this sentence.

You probably don't want to install this package alone, but amule or
amule-daemon instead., while my suggestion was:
You probably don't want to install this package alone, but amule,
amule-daemon or amule-utils-gui instead.
I think it makes sense to suggest amule-utils-gui there, as it can be
installed independently of any other of these alternatives.


I can be wrong, please me correct me if that's the case.

Best regards,
Pitxyoki

On Tue, Jul 13, 2010 at 10:21 AM, Giuseppe Iuculano iucul...@debian.orgwrote:


 tags 498693 + pending
 thanks

 Hello,

 The following change has been committed for this bug by
  Giuseppe Iuculano iucul...@debian.org on Tue, 13 Jul 2010 11:18:23
 +0200.
  The fix will be in the next upload.
 =
 Install skins files

 Closes: #498693
 Thanks: Luís Picciochi Oliveira
 =

 You can check the diff of the fix at:

http://git.debian.org/?p=pkg-amule/amule.git;a=commitdiff;h=60136ec







Bug#498693: [60136ec] Fix for Bug#498693 committed to git

2010-07-13 Thread Giuseppe Iuculano

tags 498693 + pending
thanks

Hello,

 The following change has been committed for this bug by
 Giuseppe Iuculano iucul...@debian.org on Tue, 13 Jul 2010 11:18:23 +0200.
 The fix will be in the next upload. 
=
Install skins files

Closes: #498693
Thanks: Luís Picciochi Oliveira
=

You can check the diff of the fix at:

http://git.debian.org/?p=pkg-amule/amule.git;a=commitdiff;h=60136ec





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