Bug 4349 - [diskdrake] remove the "make a backup first" dialog box
Summary: [diskdrake] remove the "make a backup first" dialog box
Status: NEW
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Low enhancement
Target Milestone: ---
Assignee: Thierry Vignaud
QA Contact:
URL:
Whiteboard: MGA8TOO
Keywords: PATCH, Triaged, USABILITY
Depends on:
Blocks:
 
Reported: 2012-01-30 20:32 CET by Barry Jackson
Modified: 2022-01-30 14:02 CET (History)
6 users (show)

See Also:
Source RPM: drakxtools-13.79-1.mga2.src.rpm
CVE:
Status comment:


Attachments
PCLOS diff to remove dialog (1.08 KB, patch)
2012-03-07 00:40 CET, Barry Jackson
Details | Diff
drakxtools-14.33-pcl-remove_diskdrake_warning-hd_gtk.pm.patch (1.05 KB, patch)
2012-08-20 00:00 CEST, Barry Jackson
Details | Diff
spec diff (683 bytes, text/plain)
2012-08-20 00:03 CEST, Barry Jackson
Details

Description Barry Jackson 2012-01-30 20:32:22 CET
Description of problem:
Every time we use diskdrake we have to hit "Continue" to bypass a useless dialog box telling us to make a backup first.
I have a small PCLOS installation that also uses diskdrake, and they dropped that dialog a few years ago - it's so nice to just use it without having to confirm. 
There are other utilities in mcc that can do just as much damage if used incorrectly - this is just overkill - let's dump it and start trusting Mageia users to have some sense.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
Comment 1 Manuel Hiebel 2012-01-31 12:03:05 CET
Hi, thanks for reporting this bug.
Assigned to the package maintainer.

bad that they don't have an svn or git repo ...

(Please set the status to 'assigned' if you are working on it)

Keywords: (none) => Triaged
Assignee: bugsquad => thierry.vignaud

Comment 2 Thierry Vignaud 2012-01-31 14:40:05 CET
Better having a checkbox for preventing it to popup again

Summary: Drop the "make a backup first" dialog box in diskdrake => [diskdrake] add a checkbox to the "make a backup first" dialog box
Keywords: (none) => USABILITY
Priority: Normal => Low

Comment 3 Thierry Vignaud 2012-02-01 10:06:47 CET
(In reply to comment #1)
> bad that they don't have an svn or git repo ...

also bad: I looked at their diff regarding the version they synced up;
like some people @mdv, they do not credit their changes, whom 90% were backports from our changes in later drakxtools (which we wrote when working @mdv)
Comment 4 Barry Jackson 2012-03-07 00:40:32 CET
Created attachment 1690 [details]
PCLOS diff to remove dialog

If it's any help this AFAICT is the patch that made the change in PCLOS.
Comment 5 Marja Van Waes 2012-05-26 13:07:33 CEST
Hi,

This bug was filed against cauldron, but we do not have cauldron at the moment.

Please report whether this bug is still valid for Mageia 2.

Thanks :)

Cheers,
marja

Keywords: (none) => NEEDINFO

Comment 6 Barry Jackson 2012-06-14 21:56:10 CEST
Yes - still valid.

Ping ping ;)

Whiteboard: (none) => MGA2TOO

Marja Van Waes 2012-08-18 19:34:18 CEST

CC: (none) => marja11
Keywords: NEEDINFO => (none)

Comment 7 Barry Jackson 2012-08-19 23:52:04 CEST
I tested the PCLOS change mentioned in #4 in mga3 drakxtools and it removes the dialog.
I really don't see the need for a checkbox.
Can we implement this as it is in Cauldron and review it if there are complaints?
Comment 8 Barry Jackson 2012-08-20 00:00:21 CEST
Created attachment 2658 [details]
drakxtools-14.33-pcl-remove_diskdrake_warning-hd_gtk.pm.patch
Comment 9 Barry Jackson 2012-08-20 00:03:01 CEST
Created attachment 2659 [details]
spec diff

Probably not the way to do it having read the NO PATCHES warning, but it worked for testing ;)
Manuel Hiebel 2012-08-20 16:05:45 CEST

CC: (none) => pterjan
Summary: [diskdrake] add a checkbox to the "make a backup first" dialog box => [diskdrake] remove the "make a backup first" dialog box
Keywords: (none) => PATCH

Comment 10 Marja Van Waes 2012-08-20 16:50:42 CEST
(In reply to comment #2)
> Better having a checkbox for preventing it to popup again

I agree.

Cauldron users are very different from average users, they know a lot more about Mageia and what is good or wrong to do. And they are more likely to have back ups.

Please don't remove the warning.
Comment 11 Barry Jackson 2012-08-21 00:00:35 CEST
PCLOS removed the warning in the 2010 release and I (or rather Google) can find no mention of any problems. 

You cannot use mcc without a root password, so you should have a clue what you are doing.

gparted does not have a warning dialog before it launches.

Don't forget that you are only *launching* the application at this point, not actually doing anything with it. 
Most dangerous operations have their own confirmation boxes before actually making changes.

I could reluctantly live with a checkbox, but will it ever be implemented?

I still favour immediate complete removal - when it's gone I'm sure no one will miss it.
Comment 12 Morgan Leijström 2012-10-16 22:47:18 CEST
I am the cautious types that would like it to ask with a dialog like that before changing any system drive.

That said i do not want it to be in the way when i just want to format a stick, backup drive or other external thing.

And when i intend to format a stick it should not let me by accident damage something the system use.


So: i wish it to 
 not pup up initially,
 not when manipulating removeables,
 not when just wanting to wiew how anythingis configured

-BUT-

It SHOULD pop up for the first action on anything used by the system.
(= is in fstab, LVM, LUKS, RAID)

CC: (none) => fri

Comment 13 Rémi Verschelde 2014-07-01 20:40:39 CEST
(In reply to Barry Jackson from comment #11)
> Don't forget that you are only *launching* the application at this point,
> not actually doing anything with it. 
> Most dangerous operations have their own confirmation boxes before actually
> making changes.
> 

Actually some irreversible operations made by diskdrake have no confirmation dialogs, or some actions can't be canceled once initiated (e.g. if you click "Create" to create a partition, there is no "Cancel" button, even though you haven't done anything yet, so you're forced to create a partition).

So a warning is needed for users who don't know diskdrake, so that they know it's not just a tool to inspect what is the content of their filesystem. A checkbox to remove the warning would be nice.

CC: (none) => remi

Comment 14 Florian Hubold 2015-10-27 09:35:12 CET
Destructible options should have such a dialog box, the same way as e.g. gparted or gnome-disks handles it. E.g. for formatting something (should not matter if it's removeable or not) or repartitioning a disk it should show up.

CC: (none) => doktor5000

Comment 15 Barry Jackson 2015-10-27 12:44:25 CET
(In reply to Florian Hubold from comment #14)
> Destructible options should have such a dialog box, the same way as e.g.
> gparted or gnome-disks handles it. E.g. for formatting something (should not
> matter if it's removeable or not) or repartitioning a disk it should show up.

Exactly, but not to initially launch the application GUI.
Comment 16 Mauricio Andrés Bustamante Viveros 2018-03-11 06:01:19 CET
Pinging Suggest

To get ride the message popup i suggest:

If standart user starts the drakdisks (diskdrake), not enable the erase, expert, unmount, format for every device contained in fstab, and enable the format and unmount for removable media
If root user, enable all options

CC: (none) => neoser10

Comment 17 Morgan Leijström 2018-03-11 10:30:28 CET
1) I do not see the need for disabling expert.  I.e expert includes setting partition label, which is very useful for external media.

2) i do think a popup warning is due at least on the first action on anything that is used by the system, or the change user tries affects fstab / LVM / LUKS / RAID.  (including changing label on a mounted unit as label may be used for mounting)

Whiteboard: MGA2TOO => MGA6TOO

Comment 18 Barry Jackson 2022-01-30 12:44:36 CET
 Wishing this bug ...

 ##### Happy 10th Birthday! #####
Comment 19 Morgan Leijström 2022-01-30 13:10:39 CET
Hurray, Hurray, Hurray! ;)

Did you not this bug in your calendar??

I understand we need to protect users from shooting themselves in the foot with this GUI, but that there should also be possible to do things on external media while protecting users from formatting wrong drive or accidentally updating fstab to include it, which breaks future boots.

We now have isodumper which in part does this, it can i.e format external USB, but it is less known to users, so would be good to have linked from panel in MCC.
I would also like the upgraded interface 
Bug 27744 - Current isodumper gui could use some improvements

I now issued:
Bug 29966 - Add isodumper to Mageia Control Center

See Also: (none) => https://bugs.mageia.org/show_bug.cgi?id=29966
Whiteboard: MGA6TOO => MGA8TOO

Comment 20 Barry Jackson 2022-01-30 13:26:09 CET
I think that Thierry's suggestion in comment 2 was perfectly adequate.

@Morgan no it was not in my calendar, but I noticed it a week ago and thought it was fitting to commemorate the occasion!
Comment 21 Morgan Leijström 2022-01-30 14:02:26 CET
Yep checkbox sounds fine to me.

Note You need to log in before you can comment on or make changes to this bug.