From: Phillip Susi <psusi@ubuntu.com>
To: "Lukáš Czerner" <lczerner@redhat.com>
Cc: Andreas Dilger <adilger@dilger.ca>, linux-ext4@vger.kernel.org
Subject: Re: [PATCH] mke2fs: don't interact with a non tty
Date: Wed, 19 Mar 2014 09:35:27 -0400 [thread overview]
Message-ID: <53299D1F.6020307@ubuntu.com> (raw)
In-Reply-To: <alpine.LFD.2.00.1403191217001.2119@localhost.localdomain>
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 3/19/2014 7:26 AM, Lukáš Czerner wrote:
> Yes, it is inconsistent especially in the way that mke2fs is
> proceeding without any problem on the device which already
> contains a valid file system (or any other) signature. Which I
> think we should really change. The problem is that this will break
> scripts for everybody which is bad.
>
> So my idea was to implement the signature check and then skip it
> if we do not have a tty attached. Just to avoid the breakage.
>
> However I do not think that we can just blindly ignore the checks
> we already have in place in the case that there is no user. But I
> agree that current behaviour is wrong and it should be changed,
> however I think that we need to change it the other way, the
> default should be no - do not proceed and exit. Because believe it
> or not, people make mistakes.
Then you are right back to breaking scripts. And yes, people make
mistakes... and unix *lets* them. You don't see rm stopping every
time you try to delete a file and saying really? *That* file? Are
you sure? You don't see dd or shell redirection stopping to ask you
if you really meant to overwrite that disk or file. There is a
*reason* why you are supposed to double check commands you are running
as root.
And putting a filesystem in an image file is one of the *least*
dangerous things you could do. Of all of the things to second guess,
and especially to default to "HALT! ERROR!" behavior, this has to be
the silliest.
If you can't assume that an interactive user knows what they are doing
and meant what they said, then at least you should assume that a
script writer knows wtf they are doing without asking them to add lots
of silly --yes-i-know-what-im-doing-stop-annoying-me flags.
> Agreed, but it should not be lifted, but rather changes to check
> for signatures on the device. The same way as it is done for
> example in xfs, or btrfs.
*NO*! Those tools annoy the hell out of me because they do that. I
*know* there is another filesystem there already ( as there are on
most disks not fresh from the factory ), why do you think I'm telling
you to change it? Do what I asked and stop treating me like a child.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.17 (MingW32)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/
iQEcBAEBAgAGBQJTKZ0fAAoJEI5FoCIzSKrwaFQH/RO0NPl2mPAlpaK2YQysegub
u80nYpSlpHjiIOLU7RCECakfELIFp1skg7lRsFdL1zLNkor4JkwW8UbOuy75WbS3
+XPAQ/1wxPzsn0J4+QM3PE3X/IZ4NWRMepl0pozpoLine87mL6u6+em2n1r1vsQK
HE/1Ma/8jqPPMXPNFDw0LMiYGyAHITfQA4c/FRwlWCbhMt2lG8dsGA7bKl7VCB5D
gmkzUF/KbgmY8xnDiIbmSHQbaF+xrIbZl8FGgi4r3CuiGZ2yZBBbs2sTCk6pvIq7
rrTQGwxBuzxaas2h+ZpLfTeFulBlIH2B+ueStinc2Br+TmBeqIKKOQ1+HjO3v6g=
=6eGY
-----END PGP SIGNATURE-----
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2014-03-19 13:35 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-18 17:11 [PATCH] mke2fs: don't interact with a non tty Phillip Susi
2014-03-18 18:31 ` Andreas Dilger
2014-03-18 18:47 ` Phillip Susi
2014-03-19 11:26 ` Lukáš Czerner
2014-03-19 13:35 ` Phillip Susi [this message]
2014-03-19 15:16 ` Lukáš Czerner
2014-03-19 16:04 ` Phillip Susi
2014-03-19 17:05 ` Lukáš Czerner
2014-03-19 17:37 ` Phillip Susi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=53299D1F.6020307@ubuntu.com \
--to=psusi@ubuntu.com \
--cc=adilger@dilger.ca \
--cc=lczerner@redhat.com \
--cc=linux-ext4@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.