[Bug 363958] New: yast2-packager needs to require yast2-*-pkg and handling of missing pkg widget

newer
[Bug 363864] New: mkbootdisk fixed...

older
[Bug 364849] New: yast2 sw_single...

bugzilla_noreply＠novell.com

22 Feb 2008 22 Feb '08

10:32

https://bugzilla.novell.com/show_bug.cgi?id=363958 Summary: yast2-packager needs to require yast2-*-pkg and handling of missing pkg widget Product: openSUSE 11.0 Version: Alpha 2 Platform: Other OS/Version: Other Status: NEW Severity: Normal Priority: P5 - None Component: YaST2 AssignedTo: lslezak@novell.com ReportedBy: msvec@novell.com QAContact: jsrain@novell.com CC: visnov@novell.com Found By: --- To prevent broken yast2-packager experience it seems we need several things: - yast2-packager needs to require yast2_pkg (virtual capability) to ensure it could run at least some way, it would be provided by all yast2-*-pkg - yast2-qt-pkg needs to require yast2-qt and ncurses the same way, to ensure at least some level of consistency - yast2-packager needs to check for the widget of the same UI it's running in and warn the user if it's not available (*), to prevent situation with running yast2-qt and having only yast2-ncurses-pkg installed (*) Ideally, it should be able to install the widget at runtime and possibly continue to run. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Show replies by date

bugzilla_noreply＠novell.com

22 Feb 22 Feb

14:36

New subject: [Bug 363958] yast2-packager needs to require yast2-*-pkg and handling of missing pkg widget

https://bugzilla.novell.com/show_bug.cgi?id=363958 User kmachalkova@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=363958#c1 Katarina Machalkova changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kmachalkova@novell.com --- Comment #1 from Katarina Machalkova 2008-02-22 07:36:12 MST --- 1) is a good point, I'm going to add it to ncurses-pkg right away. 2) is already done in nc-pkg (Requires: yast2-ncurses >= 2.16.15), but can't see it in qt-pkg 3) is not that trivial: I've fixed yast2-ncurses to throw UI Exception (and display evil red error box) if the below ReplaceWidget in inst_packages.ycp client fails: UI::ReplaceWidget(`rep, mode == nil ? `PackageSelector(`id(`packages ), "") ... It should fail, as `PackageSelector widget is not created if we don't have the appropriate plugin (YUI::widgetFactory()->createPackageSelector() returns NULL) Now it is up to ycp client to handle the failure and do not call UI::RunPkgSelection, if we don't have PackageSelector widget, possibly offer user to install suitable *-pkg package. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

bugzilla_noreply＠novell.com

14:38

New subject: [Bug 363958] yast2-packager needs to require yast2-*-pkg and handling of missing pkg widget

https://bugzilla.novell.com/show_bug.cgi?id=363958 User kmachalkova@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=363958#c2 --- Comment #2 from Katarina Machalkova 2008-02-22 07:38:50 MST --- . and for the latter, we could possibly extend each UI's GetDisplayInfo function to return a string (either of yast2-ncurses-pkg|yast2-qt-pkg|yast2-gtk-pkg) to inform YCP client which package with pkg-selector plugin it needs. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

bugzilla_noreply＠novell.com

25 Feb 25 Feb

12:02

New subject: [Bug 363958] yast2-packager needs to require yast2-*-pkg and handling of missing pkg widget

https://bugzilla.novell.com/show_bug.cgi?id=363958 User lslezak@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=363958#c3 Ladislav Slezak changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |sh@novell.com Status|NEW |ASSIGNED --- Comment #3 from Ladislav Slezak 2008-02-25 05:02:11 MST --- To comment #1 3): The package selector is a special widget and there already is UI::HasSpecialWidget() function, it should be also used for checking whether the package selector is available. What do you think? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

bugzilla_noreply＠novell.com

12:20

New subject: [Bug 363958] yast2-packager needs to require yast2-*-pkg and handling of missing pkg widget

https://bugzilla.novell.com/show_bug.cgi?id=363958 User kmachalkova@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=363958#c4 --- Comment #4 from Katarina Machalkova 2008-02-25 05:20:47 MST --- UI::HasSpecialWidget() is query-only function, it does not really create the widgets, it only questions particular UI whether it has capability to create certain widget. So you would not really find out whether `PackageSelector can be created or not. Moreover, `PackageSelector is not an optional (special) widget ;-) and imho it should not become one, because every UI should support it and implement it. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

bugzilla_noreply＠novell.com

12:59

New subject: [Bug 363958] yast2-packager needs to require yast2-*-pkg and handling of missing pkg widget

https://bugzilla.novell.com/show_bug.cgi?id=363958 User sh@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=363958#c5 --- Comment #5 from Stefan Hundhammer 2008-02-25 05:59:59 MST --- The package selector is a mandatory widget. The pattern selector and the simple patch selector, however, are optional widgets. Anyway, we will probably need a meta-provides like yast2_ui_pkg (similar to yast2_ui which yast2-qt, yast2-ncurses, and yast2-gtk provide) and make yast2-qt-pkg, yast2-ncurses-pkg, and yast2-gtk (and if/when the package part is split off from yast2-gtk yast2-gtk-pkg instead) provide that symbol and yast2-packager require it. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

bugzilla_noreply＠novell.com

13:46

New subject: [Bug 363958] yast2-packager needs to require yast2-*-pkg and handling of missing pkg widget

https://bugzilla.novell.com/show_bug.cgi?id=363958 User sh@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=363958#c6 Stefan Hundhammer changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|lslezak@novell.com |mmeeks@novell.com Status|ASSIGNED |NEW --- Comment #6 from Stefan Hundhammer 2008-02-25 06:46:06 MST --- yast2-qt-pkg now provides yast2_ui_pkg yast2-ncurses-pkg now provides yast2_ui_pkg yast2-packager now requires yast2_ui_pkg yast2-qt-pkg now requires yast2-qt yast2-ncurses-pkg already required yast2-ncurses submitted packages - yast2-qt-pkg - yast2-ncurses-pkg - yast2-packager TO DO (-> mmeeks): yast2-gtk should provide yast2_ui_pkg (until we'll get yast2-gtk-pkg, at which point yast2-gtk-pkg should provide yast2_ui_pkg) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

bugzilla_noreply＠novell.com

15:06

New subject: [Bug 363958] yast2-packager needs to require yast2-*-pkg and handling of missing pkg widget

https://bugzilla.novell.com/show_bug.cgi?id=363958 User mmeeks@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=363958#c7 --- Comment #7 from Michael Meeks 2008-02-25 08:06:14 MST --- Please don't feel that gtk+ is somehow different from qt/ncurses :-) do feel free to commit the (I assume fairly trivial) .spec file fix to yast2-gtk at the same time. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

bugzilla_noreply＠novell.com

15:26

New subject: [Bug 363958] yast2-packager needs to require yast2-*-pkg and handling of missing pkg widget

https://bugzilla.novell.com/show_bug.cgi?id=363958 User sh@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=363958#c8 Stefan Hundhammer changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED --- Comment #8 from Stefan Hundhammer 2008-02-25 08:26:12 MST --- OK. Done and submitted (yast2-gtk-2.16.5). -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

bugzilla_noreply＠novell.com

15:30

New subject: [Bug 363958] yast2-packager needs to require yast2-*-pkg and handling of missing pkg widget

https://bugzilla.novell.com/show_bug.cgi?id=363958 User visnov@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=363958#c9 --- Comment #9 from Stanislav Visnovsky 2008-02-25 08:30:13 MST --- The fix just ensures there is at least one working package manager widget. But YaST will still fail silently if user is not running the particular working UI. Or am I missing something? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

bugzilla_noreply＠novell.com

15:42

New subject: [Bug 363958] yast2-packager needs to require yast2-*-pkg and handling of missing pkg widget

https://bugzilla.novell.com/show_bug.cgi?id=363958 User sh@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=363958#c10 --- Comment #10 from Stefan Hundhammer 2008-02-25 08:42:09 MST --- (In reply to comment #9 from Stanislav Visnovsky)

...

The fix just ensures there is at least one working package manager widget.

Right. But that's more than we had before.

...

But YaST will still fail silently if user is not running the particular working UI.

It fails, but it shouldn't fail silently. You should get a YUIException that will result in an error dialog (http://svn.opensuse.org/svn/yast/trunk/ycp-ui-bindings/src/YCPErrorDialog.cc). If that doesn't work, this is another bug that needs to be fixed. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

bugzilla_noreply＠novell.com

15:53

New subject: [Bug 363958] yast2-packager needs to require yast2-*-pkg and handling of missing pkg widget

https://bugzilla.novell.com/show_bug.cgi?id=363958 User kmachalkova@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=363958#c11 --- Comment #11 from Katarina Machalkova 2008-02-25 08:53:56 MST --- Created an attachment (id=196958) --> (https://bugzilla.novell.com/attachment.cgi?id=196958) Not so silent ;-) failure to load pkg plugin Does this look any better? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

bugzilla_noreply＠novell.com

16:06

New subject: [Bug 363958] yast2-packager needs to require yast2-*-pkg and handling of missing pkg widget

https://bugzilla.novell.com/show_bug.cgi?id=363958 User sh@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=363958#c12 Stefan Hundhammer changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|FIXED | --- Comment #12 from Stefan Hundhammer 2008-02-25 09:06:47 MST --- Duh. It did exit(1) if there was no plug-in. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

bugzilla_noreply＠novell.com

16:06

New subject: [Bug 363958] yast2-packager needs to require yast2-*-pkg and handling of missing pkg widget

https://bugzilla.novell.com/show_bug.cgi?id=363958 Stefan Hundhammer changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|mmeeks@novell.com |sh@novell.com Status|REOPENED |ASSIGNED -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

bugzilla_noreply＠novell.com

16:09

New subject: [Bug 363958] yast2-packager needs to require yast2-*-pkg and handling of missing pkg widget

https://bugzilla.novell.com/show_bug.cgi?id=363958 User sh@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=363958#c13 Stefan Hundhammer changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution| |FIXED --- Comment #13 from Stefan Hundhammer 2008-02-25 09:09:38 MST --- Now throwing a proper exception that displays an error pop-up. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

bugzilla_noreply＠novell.com

29 Feb 29 Feb

14:56

New subject: [Bug 363958] yast2-packager needs to require yast2-*-pkg and handling of missing pkg widget

https://bugzilla.novell.com/show_bug.cgi?id=363958 User chrubis@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=363958#c14 Cyril Hrubis changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |casualprogrammer@yahoo.com --- Comment #14 from Cyril Hrubis 2008-02-29 07:56:15 MST --- *** Bug 364849 has been marked as a duplicate of this bug. *** https://bugzilla.novell.com/show_bug.cgi?id=364849 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

5907

Age (days ago)

5914

Last active (days ago)

List overview

Download

15 comments

1 participants

participants (1)

bugzilla_noreply＠novell.com

[Bug 363958] New: yast2-packager needs to require yast2-*-pkg and handling of missing pkg widget

tags

participants (1)