After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 571234 - Migrate from GnomeDruid to GtkAssistent
Migrate from GnomeDruid to GtkAssistent
Status: RESOLVED FIXED
Product: gnome-system-tools
Classification: Deprecated
Component: general
CVS latest
Other Linux
: Normal normal
: ---
Assigned To: Milan Bouchet-Valat
Carlos Garnacho
: 463239 (view as bug list)
Depends on:
Blocks: 582783
 
 
Reported: 2009-02-10 22:20 UTC by André Klapper
Modified: 2009-07-25 22:51 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Kill the boot tool (265.27 KB, patch)
2009-07-25 19:51 UTC, Stéphane Démurget
committed Details | Review

Description André Klapper 2009-02-10 22:20:34 UTC
See
http://www.mail-archive.com/evolution-hackers@gnome.org/msg01635.html
http://library.gnome.org/devel/gtk/stable/gtk-migrating-GtkAssistant.html
http://live.gnome.org/LibgnomeMustDie


./gnome-system-tools/src/boot/boot-druid.c: gnome_druid_set_buttons_sensitive (druid->druid, TRUE, enabled, TRUE, FALSE);
./gnome-system-tools/src/boot/boot-druid.c: gnome_druid_set_page (druid, next_page);
./gnome-system-tools/src/boot/boot-druid.c: gnome_druid_set_buttons_sensitive (druid->druid, TRUE, enabled, TRUE, FALSE);
./gnome-system-tools/src/boot/boot-druid.c: gnome_druid_set_page (druid, next_page);
./gnome-system-tools/src/boot/boot-druid.c: gnome_druid_set_buttons_sensitive (druid->druid, TRUE, enabled, TRUE, FALSE);
./gnome-system-tools/src/boot/boot-druid.c: gnome_druid_set_page (druid, next_page);
./gnome-system-tools/src/boot/boot-druid.c: gnome_druid_set_page (druid, next_page);
Comment 1 Cosimo Cecchi 2009-02-24 21:57:55 UTC
*** Bug 463239 has been marked as a duplicate of this bug. ***
Comment 2 Simon Kågedal 2009-02-24 22:30:26 UTC
Is the "boot" module ever even built? How?
Comment 3 André Klapper 2009-03-01 15:44:00 UTC
*shrug*. And I don't expect many answers here, as g-s-t is currently unmaintained. Maybe another option is to simply drop that part of the code if it's really unused.
Comment 4 André Klapper 2009-04-27 23:32:18 UTC
Simon:

(In reply to comment #2)
> Is the "boot" module ever even built? How?

According to bug 572355 comment 14:
"The code in /src/boot is unused and not even built."
Comment 5 Stéphane Démurget 2009-07-18 23:09:15 UTC
Indeed in the following commit, the boot and disk tool were removed:

http://git.gnome.org/cgit/gnome-system-tools/commit/?id=5660bf86ed2f650f7606be0c3a31f9f9d5fdcd75
Comment 6 André Klapper 2009-07-19 02:08:34 UTC
Well, the code was not removed from the repo. It's just that the build options for them were removed.

Now who is brave enough (and maintainer) to remove the files from the git repository? People interested in it can still use an older revision...
Comment 7 Stéphane Démurget 2009-07-19 09:38:47 UTC
Yes, I was just indicating there is not even a configuration option to build them.

From http://bugzilla.gnome.org/show_bug.cgi?id=582783#c2, it seems Andy might be brave enough :p

Anyway it has always been marked experimental and Ubuntu & Debian, the only major users of g-s-t these days do not ship it since a long time.

I can make a quick patch if somebody is really interested and wants to close this quickly. Well we could close this one and #582783 anyway since this one is a false positive.

Carlos: your b.g.o. address seems wrong (or you have some redirection from imendio maybe). Could you update it? Do you know any distro using the boot tool and do you think it's worth porting it?
Comment 8 Milan Bouchet-Valat 2009-07-19 10:01:51 UTC
Please post patch if you can, I'll commit it. Removing not used code is not a problem to me... Thanks!
Comment 9 Stéphane Démurget 2009-07-19 10:36:57 UTC
Ok, I added the flags to build, but on top of that:
- there is not even any tiny support in liboobs/system-tools-backend
- it does not compile because of gnome-druid but glade too

I was starting to retrieve the glade file from history and all, but since it will be removed afterwards and will still not work without the backend...

Forget it, just trash the boot/ directory and that's all. We close both bugs, thanks.
Comment 10 André Klapper 2009-07-24 18:15:25 UTC
So, can someone go ahead and remove the code from git, please? Highly welcome.
Comment 11 Stéphane Démurget 2009-07-25 19:51:25 UTC
Created attachment 139210 [details] [review]
Kill the boot tool

Ho, I thought Milan would directly remove the directories. If you really need a patch, here it is.
Comment 12 Milan Bouchet-Valat 2009-07-25 22:51:50 UTC
Thanks! I could remove the directories, but I would have missed the .spec.in file, so your patch was useful. Another goal reached for the release! Please go on providing patches, I'll look after them...

Pushed as de6a626edcc34664480f1ffe7a363c4c6eed3f19.