Bug#582884: [Usb-creator-hackers] Bug#582884: ITP: usb-creator -- Live USB creator

2010-06-03 Thread Evan Dandrea
On Wed, Jun 2, 2010 at 4:36 PM, Alexander Reichle-Schmehl
toli...@debian.org wrote:
 Am 27.05.2010 13:47, schrieb Evan Dandrea:
 I can't say I agree, but if we're going to have to do this, we might
 as well take the opportunity to make it consistent with the .desktop
 file.  Alexander, are you okay with startup-disk-creator?

 Yes, that sounds good to me.

Great, will do!  One question though.  Are you requiring that we
change the name of the source package as well as the binary packages?

Thanks



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



Bug#582884: [Usb-creator-hackers] Bug#582884: ITP: usb-creator -- Live USB creator

2010-05-27 Thread Evan Dandrea
On Thu, May 27, 2010 at 12:30 PM, Ignace Mouzannar mouzan...@gmail.com wrote:
 On Thu, May 27, 2010 at 12:46, Alexander Reichle-Schmehl
 toli...@debian.org wrote:
 Yes, the name is too generic.
 No, it is already in Ubuntu is no argument.
 No, it will lead to some kind of problems for Ubuntu to have it in
 Debian with a different way) is also not an argument.
 Yes, we have and will reject packages because of too generic names
 (several gnustep package come to my mind).

 Here are some alternative names:
  - live-usb-creator (as Mehdi suggested earlier)
  - iso-usb-creator
  - startup-disk-creator (this is what the Gnome menu item reads [1])
  - startup-usb-creator
  - ubuntu-usb-creator

I can't say I agree, but if we're going to have to do this, we might
as well take the opportunity to make it consistent with the .desktop
file.  Alexander, are you okay with startup-disk-creator?

I think if we can get rid of usb in the name, that would be good as it
can write to SD cards.  I want to avoid using live in the name since,
as I previously mentioned, it can write debian-installer based images,
which are definitely not live CDs.  I don't want to put Ubuntu in the
name.  It writes more than just Ubuntu images (currently any
Debian-based image, but in the future any Linux distribution that uses
GRUB2).



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



Bug#576359: [Usb-creator-hackers] Bug#576359: Bug#582884: ITP: usb-creator -- Live USB creator

2010-05-26 Thread Evan Dandrea
On Wed, May 26, 2010 at 12:42 AM, Ignace Mouzannar mouzan...@gmail.com wrote:
 Excellent job. Howerver, it seems that the version 0.2.23 has not been
 released by upstream yet. I prefer waiting for the new release before
 trying to upload it to Debian.

This is a native package.  0.2.23 will be released when it's uploaded :).

 The maintainer is currently set as Ubuntu Installer Team as these
 Ubuntu  Debian Developers have actually created and maintained this
 package.

Actually, this is a bug.  It should be the usb-creator-hackers team now.  Fixed.

 As we will be maintaining this package for Debian, we will set
 ourselves in the Maintainer and Uploaders fields.

Is this necessary?  I'm more than happy to add people to the
usb-creator-hackers team.  I'd really like to avoid having a delta
between Debian and Ubuntu on this, if possible.

  * .desktop files:
   - I am not convinced that usb-creator's Desktop file should appear
 in System/Administration as using it does not have an effect on the
 local system. I'd rather put it in Application/System tools.
     What do you think about this?

It has an effect on the devices attached to the local system.  There
are other applications in System-Administration that are similar in
nature to usb-creator, such as GParted and Disk Utility (palimpsest).

Does usb-creator not appear in the same menu as these two applications
in Debian?  You don't specify that something goes into the
Administration menu in the XDG spec, but rather list a set of
categories it falls under.  So if you're seeing different behavior for
similar programs, we might have the set of categories wrong.

Thanks!



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



Bug#582884: [Usb-creator-hackers] Bug#582884: ITP: usb-creator -- Live USB creator

2010-05-24 Thread Evan Dandrea
On Mon, May 24, 2010 at 12:43 PM, Mehdi Dogguy me...@dogguy.org wrote:
 * Package name    : usb-creator

 usb-creator is a bit misleading (or at least… not clear). Could
 you rename it into something like live-usb-creator?
 This package has been shipped in Ubuntu for a few releases now.

 Honestly, I don't know how you (as a team) ended up with such a name.

I do not think it is misleading at all.  Calling it live-usb-creator
would just add confusion as it can be used to write debian-installer
ISOs as well, which clearly are not live CDs.

 I don't believe the name is misleading when read together with short
 description or with long description.

 Having a clear short description is not a valid reason to keep that
 name, IMO. And such a name may not be appropriate because it's too
 general. live-usb-creator tells you what the program does, without
 reading its short description.

I don't think you can encapsulate what this program does in the
package name alone.

Equally, Evolution isn't packaged as evolution-mail-client, despite
evolution telling you nothing about what it does.  So why should we
apply this new rule to just usb-creator?



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



Bug#542635: mcopy does not respect input from /dev/null

2009-08-20 Thread Evan Dandrea
Package: mtools
Version: 4.0.10-1

mcopy will still ask for confirmation, even if you pipe /dev/null into it.

The attached patch fixes the issue for me, but I am not sure of
whether it may have other implications.
Index: mtools-4.0.10/tty.c
===
--- mtools-4.0.10.orig/tty.c	2009-08-20 15:33:41.0 +0100
+++ mtools-4.0.10/tty.c	2009-08-20 15:45:34.0 +0100
@@ -171,6 +171,10 @@
 {
 	if(notty)
 		return NULL;
+	if ( !isatty(0) ){
+		notty = 1;
+		return NULL;
+	}
 	if (tty == NULL) {
 		ttyfd = open(/dev/tty, O_RDONLY);
 		if(ttyfd = 0) {
@@ -178,10 +182,6 @@
 		}
 	}
 	if  (tty == NULL){
-		if ( !isatty(0) ){
-			notty = 1;
-			return NULL;
-		}
 		ttyfd = 0;
 		tty = stdin;
 	}