October 2008 - YaST Commits - openSUSE Mailing Lists

[yast-commit] r51884 - in /trunk/tune: VERSION hwinfo/src/newid.ycp package/yast2-tune.changes
by lslezak＠svn.opensuse.org 06 Oct '08

06 Oct '08

Author: lslezak Date: Mon Oct 6 18:18:52 2008 New Revision: 51884 URL: http://svn.opensuse.org/viewcvs/yast?rev=51884&view=rev Log: - 'Kernel Settings' - align the action buttons to the left (bnc#419535) - 2.17.5 Modified: trunk/tune/VERSION trunk/tune/hwinfo/src/newid.ycp trunk/tune/package/yast2-tune.changes Modified: trunk/tune/VERSION URL: http://svn.opensuse.org/viewcvs/yast/trunk/tune/VERSION?rev=51884&r1=51883&… ============================================================================== --- trunk/tune/VERSION (original) +++ trunk/tune/VERSION Mon Oct 6 18:18:52 2008 @@ -1 +1 @@ -2.17.4 +2.17.5 Modified: trunk/tune/hwinfo/src/newid.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/tune/hwinfo/src/newid.ycp?rev=51… ============================================================================== --- trunk/tune/hwinfo/src/newid.ycp (original) +++ trunk/tune/hwinfo/src/newid.ycp Mon Oct 6 18:18:52 2008 @@ -340,7 +340,8 @@ `HSpacing(1), `PushButton(`id(`edit), Label::EditButton()), `HSpacing(1), - `PushButton(`id(`delete), Label::DeleteButton()) + `PushButton(`id(`delete), Label::DeleteButton()), + `HStretch() ), `VSpacing(0.5) ); Modified: trunk/tune/package/yast2-tune.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/tune/package/yast2-tune.changes?… ============================================================================== --- trunk/tune/package/yast2-tune.changes (original) +++ trunk/tune/package/yast2-tune.changes Mon Oct 6 18:18:52 2008 @@ -1,4 +1,11 @@ ------------------------------------------------------------------- +Mon Oct 6 18:17:05 CEST 2008 - lslezak(a)suse.cz + +- 'Kernel Settings' - align the action buttons to the left + (bnc#419535) +- 2.17.5 + +------------------------------------------------------------------- Mon Oct 6 13:32:22 CEST 2008 - visnov(a)suse.cz - adapt to new Confirm::Detection API (bnc #431276) -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r51883 - in /trunk/autoinstallation: package/autoyast2.changes src/clients/autoyast.ycp src/clients/software_auto.ycp
by lslezak＠svn.opensuse.org 06 Oct '08

06 Oct '08

Author: lslezak Date: Mon Oct 6 17:57:54 2008 New Revision: 51883 URL: http://svn.opensuse.org/viewcvs/yast?rev=51883&view=rev Log: - fixed some more GPG callbacks (see bnc#370223) Modified: trunk/autoinstallation/package/autoyast2.changes trunk/autoinstallation/src/clients/autoyast.ycp trunk/autoinstallation/src/clients/software_auto.ycp Modified: trunk/autoinstallation/package/autoyast2.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/autoinstallation/package/autoyas… ============================================================================== --- trunk/autoinstallation/package/autoyast2.changes (original) +++ trunk/autoinstallation/package/autoyast2.changes Mon Oct 6 17:57:54 2008 @@ -1,4 +1,9 @@ ------------------------------------------------------------------- +Mon Oct 6 17:57:04 CEST 2008 - lslezak(a)suse.cz + +- fixed some more GPG callbacks (see bnc#370223) + +------------------------------------------------------------------- Fri Sep 26 17:12:32 CEST 2008 - ug(a)suse.de - optimization for package evaluation removed. Modified: trunk/autoinstallation/src/clients/autoyast.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/autoinstallation/src/clients/aut… ============================================================================== --- trunk/autoinstallation/src/clients/autoyast.ycp (original) +++ trunk/autoinstallation/src/clients/autoyast.ycp Mon Oct 6 17:57:54 2008 @@ -35,10 +35,10 @@ return `abort; } - Pkg::CallbackImportGpgKey(AutoInstall::callbackTrue_boolean_map); - Pkg::CallbackAcceptUnknownGpgKey(AutoInstall::callbackTrue_boolean_string_string); + Pkg::CallbackImportGpgKey(AutoInstall::callbackTrue_boolean_map_integer); + Pkg::CallbackAcceptUnknownGpgKey(AutoInstall::callbackTrue_boolean_string_string_integer); Pkg::CallbackAcceptFileWithoutChecksum(AutoInstall::callbackTrue_boolean_string); - Pkg::CallbackAcceptUnsignedFile(AutoInstall::callbackTrue_boolean_string); + Pkg::CallbackAcceptUnsignedFile(AutoInstall::callbackTrue_boolean_string_integer); define boolean openFile( map <string, string> options ) { if( !Profile::ReadXML( options["filename"]:"" ) ) { Modified: trunk/autoinstallation/src/clients/software_auto.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/autoinstallation/src/clients/sof… ============================================================================== --- trunk/autoinstallation/src/clients/software_auto.ycp (original) +++ trunk/autoinstallation/src/clients/software_auto.ycp Mon Oct 6 17:57:54 2008 @@ -53,7 +53,7 @@ "); //Pkg::TargetFinish (); Pkg::CallbackAcceptFileWithoutChecksum(AutoInstall::callbackTrue_boolean_string); - Pkg::CallbackAcceptUnsignedFile(AutoInstall::callbackTrue_boolean_string); + Pkg::CallbackAcceptUnsignedFile(AutoInstall::callbackTrue_boolean_string_integer); Pkg::TargetInit("/", false); // AutoinstSoftware::pmInit(); -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r51882 - /tags/stable-2_17_5/sound/
by lslezak＠svn.opensuse.org 06 Oct '08

06 Oct '08

Author: lslezak Date: Mon Oct 6 17:23:23 2008 New Revision: 51882 URL: http://svn.opensuse.org/viewcvs/yast?rev=51882&view=rev Log: Created tag stable-2_17_5 for sound Added: tags/stable-2_17_5/sound/ - copied from r51881, trunk/sound/ -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r51881 - in /trunk/sound: VERSION package/yast2-sound.changes sound/src/Sound.ycp sound/src/write_routines.ycp
by lslezak＠svn.opensuse.org 06 Oct '08

06 Oct '08

Author: lslezak Date: Mon Oct 6 17:22:35 2008 New Revision: 51881 URL: http://svn.opensuse.org/viewcvs/yast?rev=51881&view=rev Log: - fixed storing additional module parameters (bnc#404900) (check the key before removal from a map) - 2.17.5 Modified: trunk/sound/VERSION trunk/sound/package/yast2-sound.changes trunk/sound/sound/src/Sound.ycp trunk/sound/sound/src/write_routines.ycp Modified: trunk/sound/VERSION URL: http://svn.opensuse.org/viewcvs/yast/trunk/sound/VERSION?rev=51881&r1=51880… ============================================================================== --- trunk/sound/VERSION (original) +++ trunk/sound/VERSION Mon Oct 6 17:22:35 2008 @@ -1 +1 @@ -2.17.3 +2.17.5 Modified: trunk/sound/package/yast2-sound.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/sound/package/yast2-sound.change… ============================================================================== --- trunk/sound/package/yast2-sound.changes (original) +++ trunk/sound/package/yast2-sound.changes Mon Oct 6 17:22:35 2008 @@ -1,4 +1,11 @@ ------------------------------------------------------------------- +Mon Oct 6 17:18:43 CEST 2008 - lslezak(a)suse.cz + +- fixed storing additional module parameters (bnc#404900) + (check the key before removal from a map) +- 2.17.5 + +------------------------------------------------------------------- Mon Oct 6 15:10:59 CEST 2008 - lslezak(a)suse.cz - the sound card DB has been updated to kernel-2.6.27-rc7-12 Modified: trunk/sound/sound/src/Sound.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/sound/sound/src/Sound.ycp?rev=51… ============================================================================== --- trunk/sound/sound/src/Sound.ycp (original) +++ trunk/sound/sound/src/Sound.ycp Mon Oct 6 17:22:35 2008 @@ -1129,8 +1129,11 @@ maplist (string modname, (list<string>) mods, ``{ map opts = CollectOptions(modname); - // ignore "index" options - opts = remove(opts, "index"); + if (haskey(opts, "index")) + { + // ignore "index" options + opts = remove(opts, "index"); + } string optstr = ""; maplist(string k, string v, (map<string,string>) opts, ``{ Modified: trunk/sound/sound/src/write_routines.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/sound/sound/src/write_routines.y… ============================================================================== --- trunk/sound/sound/src/write_routines.ycp (original) +++ trunk/sound/sound/src/write_routines.ycp Mon Oct 6 17:22:35 2008 @@ -264,10 +264,17 @@ // remove "index" option, do not write it string index = res["index"]:""; - res = remove(res, "index"); + + if (haskey(res, "index")) + { + res = remove(res, "index"); + } // remove old "enable" option if it's present - res = remove(res, "enable"); + if (haskey(res, "enable")) + { + res = remove(res, "enable"); + } list<string> indices = splitstring(index, ","); @@ -389,6 +396,7 @@ * @return map return struct: $["return": boolean, "err_msg": string] */ define map SaveModulesEntry (list save_info, list system) ``{ + y2milestone("SaveModulesEntry: %1, %2", save_info, system); path mod = .modprobe_sound; path mod_alias = .modprobe_sound.alias; -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r51880 - /trunk/installation/control/control.SLED.xml
by locilka＠svn.opensuse.org 06 Oct '08

06 Oct '08

Author: locilka Date: Mon Oct 6 16:44:18 2008 New Revision: 51880 URL: http://svn.opensuse.org/viewcvs/yast?rev=51880&view=rev Log: inst_congratulate used even in SLED with AC (bnc#428190#c10) Modified: trunk/installation/control/control.SLED.xml Modified: trunk/installation/control/control.SLED.xml URL: http://svn.opensuse.org/viewcvs/yast/trunk/installation/control/control.SLE… ============================================================================== --- trunk/installation/control/control.SLED.xml (original) +++ trunk/installation/control/control.SLED.xml Mon Oct 6 16:44:18 2008 @@ -87,6 +87,7 @@ <autoconfiguration_enabled_modules config:type="list"> <module>cont_suse_register</module> <module>cont_suseconfig</module> + <module>cont_congratulate</module> </autoconfiguration_enabled_modules>  @@ -952,7 +953,8 @@ </module> <module> <label>Hardware Configuration</label> - <name>congratulate</name> + <name>cont_congratulate</name> + <execute>inst_congratulate</execute> </module> </modules> </workflow> -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r51879 - in /trunk/gtk: ChangeLog src/YGDialog.cc
by michael_meeks＠svn.opensuse.org 06 Oct '08

06 Oct '08

Author: michael_meeks Date: Mon Oct 6 16:36:58 2008 New Revision: 51879 URL: http://svn.opensuse.org/viewcvs/yast?rev=51879&view=rev Log: 2008-10-06 Michael Meeks <michael.meeks(a)novell.com> * src/YGDialog.cc: set dialog a11y role on dialogs. Modified: trunk/gtk/ChangeLog trunk/gtk/src/YGDialog.cc Modified: trunk/gtk/ChangeLog URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/ChangeLog?rev=51879&r1=51878… ============================================================================== --- trunk/gtk/ChangeLog (original) +++ trunk/gtk/ChangeLog Mon Oct 6 16:36:58 2008 @@ -1,3 +1,7 @@ +2008-10-06 Michael Meeks <michael.meeks(a)novell.com> + + * src/YGDialog.cc: set dialog a11y role on dialogs. + 2008-10-02 Ricardo Cruz <rpmcruz(a)alunos.dcc.fc.up.pt> * src/yzyppwrapper.cc: must update pools only after running the solver, Modified: trunk/gtk/src/YGDialog.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/src/YGDialog.cc?rev=51879&r1… ============================================================================== --- trunk/gtk/src/YGDialog.cc (original) +++ trunk/gtk/src/YGDialog.cc Mon Oct 6 16:36:58 2008 @@ -78,6 +78,9 @@ gtk_window_set_modal (window, TRUE); gtk_window_set_transient_for (window, parent); gtk_window_set_type_hint (window, GDK_WINDOW_TYPE_HINT_DIALOG); + AtkObject *peer = gtk_widget_get_accessible (GTK_WIDGET (window)); + if (peer != NULL) + atk_object_set_role (peer, ATK_ROLE_DIALOG); } else { gtk_window_set_title (window, "YaST"); -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r51878 - in /trunk/installation: package/yast2-installation.changes src/clients/inst_proposal.ycp
by locilka＠svn.opensuse.org 06 Oct '08

06 Oct '08

Author: locilka Date: Mon Oct 6 16:27:27 2008 New Revision: 51878 URL: http://svn.opensuse.org/viewcvs/yast?rev=51878&view=rev Log: - Always change initial proposal [Next] button to [Install], resp. [Update] (bnc #431567). - 2.17.18 Modified: trunk/installation/package/yast2-installation.changes trunk/installation/src/clients/inst_proposal.ycp Modified: trunk/installation/package/yast2-installation.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/installation/package/yast2-insta… ============================================================================== --- trunk/installation/package/yast2-installation.changes (original) +++ trunk/installation/package/yast2-installation.changes Mon Oct 6 16:27:27 2008 @@ -8,6 +8,8 @@ #432572) - Dropping obsolete inst_software_selection client instead of convwerting it - not in use anymore (bnc #432572). +- Always change initial proposal [Next] button to [Install], + resp. [Update] (bnc #431567). - 2.17.18 ------------------------------------------------------------------- Modified: trunk/installation/src/clients/inst_proposal.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/installation/src/clients/inst_pr… ============================================================================== --- trunk/installation/src/clients/inst_proposal.ycp (original) +++ trunk/installation/src/clients/inst_proposal.ycp Mon Oct 6 16:27:27 2008 @@ -1129,16 +1129,16 @@ // make_proposal( false, false ); - // Set keyboard focus to the [Install] / [Update] or [Next] button - Wizard::SetFocusToNextButton(); - if (Stage::initial ()) - Wizard::SetNextButton (`next, ( - // FATE #120373 - Mode::update() ? - _("&Update") - : - _("&Install") - )); + void SetNextButton () { + if (Stage::initial () && proposal_mode == "initial") + Wizard::SetNextButton (`next, ( + // FATE #120373 + Mode::update() ? + _("&Update") + : + _("&Install") + )); + } // // Input loop @@ -1146,9 +1146,16 @@ any input = nil; + // Set keyboard focus to the [Install] / [Update] or [Next] button + Wizard::SetFocusToNextButton(); + while ( true ) { richtext_normal_cursor(`id(`proposal ) ); + // bnc #431567 + // Some proposal module can change it while called + SetNextButton(); + input = Wizard::UserInput(); y2milestone("Proposal - UserInput: '%1'", input); richtext_busy_cursor(`id(`proposal ) ); -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r51877 - /tags/stable-2_17_4/fingerprint-reader/
by jsuchome＠svn.opensuse.org 06 Oct '08

06 Oct '08

Author: jsuchome Date: Mon Oct 6 16:24:17 2008 New Revision: 51877 URL: http://svn.opensuse.org/viewcvs/yast?rev=51877&view=rev Log: Created tag stable-2_17_4 for fingerprint-reader Added: tags/stable-2_17_4/fingerprint-reader/ - copied from r51876, trunk/fingerprint-reader/ -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r51876 - in /trunk/fingerprint-reader: ./ package/ src/
by jsuchome＠svn.opensuse.org 06 Oct '08

06 Oct '08

Author: jsuchome Date: Mon Oct 6 16:24:06 2008 New Revision: 51876 URL: http://svn.opensuse.org/viewcvs/yast?rev=51876&view=rev Log: - removed Supplements from spec file (package will be recommended by pam_fp) - pam_fp used instead of pam_thinkfinger - save fingerprints in user's home directory - disabled option to import fingerprints - 2.17.4 Modified: trunk/fingerprint-reader/VERSION trunk/fingerprint-reader/package/yast2-fingerprint-reader.changes trunk/fingerprint-reader/src/FingerprintReader.ycp trunk/fingerprint-reader/src/UsersPluginFingerprintReader.pm trunk/fingerprint-reader/src/dialogs.ycp trunk/fingerprint-reader/src/users_plugin_fingerprint_reader.ycp Modified: trunk/fingerprint-reader/VERSION URL: http://svn.opensuse.org/viewcvs/yast/trunk/fingerprint-reader/VERSION?rev=5… ============================================================================== --- trunk/fingerprint-reader/VERSION (original) +++ trunk/fingerprint-reader/VERSION Mon Oct 6 16:24:06 2008 @@ -1 +1 @@ -2.17.3 +2.17.4 Modified: trunk/fingerprint-reader/package/yast2-fingerprint-reader.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/fingerprint-reader/package/yast2… ============================================================================== --- trunk/fingerprint-reader/package/yast2-fingerprint-reader.changes (original) +++ trunk/fingerprint-reader/package/yast2-fingerprint-reader.changes Mon Oct 6 16:24:06 2008 @@ -1,4 +1,14 @@ ------------------------------------------------------------------- +Mon Oct 6 16:05:44 CEST 2008 - jsuchome(a)suse.de + +- removed Supplements from spec file (package will be recommended + by pam_fp) +- pam_fp used instead of pam_thinkfinger +- save fingerprints in user's home directory +- disabled option to import fingerprints +- 2.17.4 + +------------------------------------------------------------------- Wed Oct 1 11:53:49 CEST 2008 - jsuchome(a)suse.cz - call _exit() to exit the child process, not exit() (bnc#430229) Modified: trunk/fingerprint-reader/src/FingerprintReader.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/fingerprint-reader/src/Fingerpri… ============================================================================== --- trunk/fingerprint-reader/src/FingerprintReader.ycp (original) +++ trunk/fingerprint-reader/src/FingerprintReader.ycp Mon Oct 6 16:24:06 2008 @@ -51,15 +51,15 @@ /** * Required packages for this module to operate */ -global list<string> required_packages = ["pam_thinkfinger"]; +global list<string> required_packages = ["pam_fp"]; // packages to install for specific architectures (see e.g. #224510) global map arch_packages = $[ - "pam_thinkfinger" : $[ - "ia64" : [ "pam_thinkfinger-x86" ], - "ppc64" : [ "pam_thinkfinger-64bit" ], - "s390_64" : [ "pam_thinkfinger-32bit" ], - "x86_64" : [ "pam_thinkfinger-32bit" ], + "pam_fp" : $[ + "ia64" : [ "pam_fp-x86" ], + "ppc64" : [ "pam_fp-64bit" ], + "s390_64" : [ "pam_fp-32bit" ], + "x86_64" : [ "pam_fp-32bit" ], ], ]; @@ -80,11 +80,6 @@ global string import_dir = ""; /** - * Directory for storing fingerprint files - */ -global string bir_dir = "/etc/pam_thinkfinger"; - -/** * Get the list of fingerprint readers */ global list ReadFingerprintReaderDevices () { @@ -96,7 +91,7 @@ } /** - * If pam_mount is enabled, pam_thinkfinger cannot be used (bnc#390810) + * If pam_mount is enabled, pam_fp cannot be used (bnc#390810) */ global boolean CryptedHomesEnabled () { @@ -135,11 +130,11 @@ } if (CryptedHomesEnabled ()) { - // error popup (pam_thinkfinger cannot work with pam_mount) + // error popup (pam_fp cannot work with pam_mount) Report::Error (_("Fingerprint reader devices cannot be used when encrypted directories are used.")); return false; } - use_pam = Pam::Enabled ("thinkfinger"); + use_pam = Pam::Enabled ("fp"); modified = false; return true; @@ -180,7 +175,7 @@ Progress::NextStage(); boolean pam_ret = use_pam ? - Pam::Add ("thinkfinger") : Pam::Remove ("thinkfinger"); + Pam::Add ("fp") : Pam::Remove ("fp"); if (!pam_ret) // Error message Report::Error (_("Cannot write PAM settings.")); @@ -188,6 +183,7 @@ sleep(sl); Progress::NextStage(); + /* FIXME import into users homes? if (use_pam && import_dir != "") { if (!FileUtils::Exists (bir_dir)) @@ -210,6 +206,7 @@ } } } + */ Progress::NextStage(); sleep(sl); Modified: trunk/fingerprint-reader/src/UsersPluginFingerprintReader.pm URL: http://svn.opensuse.org/viewcvs/yast/trunk/fingerprint-reader/src/UsersPlug… ============================================================================== --- trunk/fingerprint-reader/src/UsersPluginFingerprintReader.pm (original) +++ trunk/fingerprint-reader/src/UsersPluginFingerprintReader.pm Mon Oct 6 16:24:06 2008 @@ -24,7 +24,6 @@ YaST::YCP::Import ("FingerprintReader"); YaST::YCP::Import ("Pam"); YaST::YCP::Import ("SCR"); -YaST::YCP::Import ("Users"); ##-------------------------------------- ##--------------------- global variables @@ -34,9 +33,6 @@ my $fingerprint_reader_available = undef; -# path to store fingerprints -my $bir_path = "/etc/pam_thinkfinger"; - my $name = "UsersPluginFingerprintReader"; ##---------------------------------------- @@ -65,7 +61,8 @@ # helper, check if Fingerprint Reader was already configured sub fingerprint_reader_configured { - return Pam->Enabled ("thinkfinger"); +# return Pam->Enabled ("fp"); + return 1; } # helper function: check if Fingerprint Reader (the device) is available @@ -81,37 +78,38 @@ # check if given user has fingerprint authentication configured sub fingerprint_present { - my ($data) = @_; - my $username = $data->{"uid"}; - return 0 if !$username; - my $org_username = $data->{"org_user"}{"uid"} || $username; - my $org_bir_file = "$bir_path/$org_username.bir"; - return (FileUtils->Exists ($org_bir_file)); + my ($data) = @_; + my $home = $data->{"homeDirectory"}; + return 0 if !$home; + my $org_home= $data->{"org_user"}{"homeDirectory"} || $home; + return (FileUtils->IsDirectory ("$org_home/.fprint/prints")); } -# check if bir file name corresponds with username +# check if fingerprint directory corresponds with username sub adapt_fingerprint_info { my ($config, $data) = @_; my $username = $data->{"uid"}; - my $org_username = $data->{"org_user"}{"uid"} || $username; - my $bir_file = $data->{"bir_file"}; - return $data if !defined $username; - if (!defined $bir_file && $username ne $org_username) { - # When user was only renamed, but new fingerprint was not generated, - # there's no 'bir_file' entry in the map -> - # backup his original bir file to tmpdir so it won't get lost and can - # be correctly renamed in Write. - my $org_bir_file = "$bir_path/$org_username.bir"; - if (FileUtils->Exists ($org_bir_file)) { - my $tmpdir = Directory->tmpdir (); - SCR->Execute (".target.bash", "/bin/cp $org_bir_file $tmpdir/$username.bir"); - $data->{"bir_file"} = $username; - } - } - if ($bir_file && $bir_file ne $username) { - $data->{"bir_file"} = $username; + + my $fingerprint_dir = $data->{"_fingerprint_dir"}; + + # if _fingerprint_dir is not defined, plugin UI wasn't called (yet) + # and we don't need to solve user renaming, as the files will be moved + # together with a home directory + + # but we need to move temporary directory if it exists and user was renamed: + if ($fingerprint_dir && $fingerprint_dir ne $username) { + my $tmp_dir = Directory->tmpdir (); + my $new_dir = "$tmp_dir/$username"; + # there might be some directory related to deleted user - remove it: + SCR->Execute (".target.bash", "/bin/rm -rf $new_dir"); + # move the current tmp directory + SCR->Execute (".target.bash", + "/bin/mv $tmp_dir/$fingerprint_dir $new_dir"); + # reflect the renaming in the data hash + $data->{"_fingerprint_dir"} = $username; } + return $data; } @@ -119,7 +117,7 @@ sub remove_plugin_data { my ($config, $data) = @_; - $data->{"bir_file"} = ""; + $data->{"_fingerprint_dir"} = ""; $data->{"plugin_modified"} = 1; return $data; } @@ -307,48 +305,38 @@ # What should be done after user is finally written (this is called only once): -# - remove org_username.bir file, -# but only when there is no other (new) user of that name -# - move temporary bir file to correct location +# - move temporary directory with fingerprints to correct location ($HOME) BEGIN { $TYPEINFO{Write} = ["function", "boolean", "any", "any"];} sub Write { my ($self, $config, $data) = @_; my $username = $data->{"uid"}; - my $org_username = $data->{"org_user"}{"uid"} || $username; - my $bir_file = $data->{"bir_file"}; - my $org_bir_file = "$bir_path/$org_username.bir"; - - if (($config->{"modified"} || "") eq "deleted") { - # check if some new user doesn't have the name of the deleted one - my $u = Users->GetUserByName ($org_username, ""); - if (!defined $u->{"bir_file"}) { - $bir_file = ""; + my $fingerprint_dir = $data->{"_fingerprint_dir"}; + + return YaST::YCP::Boolean (0) if (!$username); + + if ($fingerprint_dir) { + my $tmp_dir = Directory->tmpdir (); + $fingerprint_dir= "$tmp_dir/$fingerprint_dir/.fprint"; + my $home = $data->{"homeDirectory"}; + if (!$home) { + y2warning ("home directory empty or not defined"); + return YaST::YCP::Boolean (0); } - } - return if (!$username || ! defined $bir_file); - if ($username ne $org_username) { - # check if there was old bir-file - my $u = Users->GetUserByName ($org_username, ""); - # do not delete birfile when some new user was added with the name - # of this one's original name - if (!defined $u->{"bir_file"} && FileUtils->Exists ($org_bir_file)) { - y2milestone ("removing old file $org_bir_file owned by $username"); - SCR->Execute (".target.bash", "/bin/rm $org_bir_file"); + my $uid = $data->{"uidNumber"}; + my $gid = $data->{"gidNumber"}; + + my $command = "/bin/chown -R $uid:$gid $fingerprint_dir"; + my %out = %{SCR->Execute (".target.bash_output", $command)}; + if (($out{"stderr"} || "") ne "") { + y2error ("error calling $command: ", $out{"stderr"} || ""); + return YaST::YCP::Boolean (0); + } + else { + y2milestone ("moving $fingerprint_dir to $home/"); + SCR->Execute (".target.bash","/bin/cp -ar $fingerprint_dir $home/"); } - } - elsif ($bir_file eq "" && FileUtils->Exists ($org_bir_file)) { - # disable fingerprint authentication for this user - # (now $org_username is same as $username) - y2milestone ("removing file $org_bir_file for user $username"); - SCR->Execute (".target.bash", "/bin/rm $org_bir_file"); - } - if ($bir_file) { - my $bir_file_path = Directory->tmpdir (); - $bir_file_path = "$bir_file_path/$bir_file.bir"; - y2milestone ("moving $bir_file_path to $bir_path/"); - SCR->Execute (".target.bash", "/bin/mv $bir_file_path $bir_path/"); } return YaST::YCP::Boolean (1); } Modified: trunk/fingerprint-reader/src/dialogs.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/fingerprint-reader/src/dialogs.y… ============================================================================== --- trunk/fingerprint-reader/src/dialogs.ycp (original) +++ trunk/fingerprint-reader/src/dialogs.ycp Mon Oct 6 16:24:06 2008 @@ -82,10 +82,12 @@ string help_text = _(" Fingerprint Authentication The fingerprint reader configuration updates your PAM settings to enable authentication with fingerprints. -") + +"); + /* // help text, cont. sformat (_("To Import Fingerprint Files from different installations, choose the directory where the original files are located. The files will be copied to directory %1."), FingerprintReader::bir_dir); + */ term con = `HBox (`HSpacing (3), `VBox ( // frame label @@ -103,7 +105,8 @@ ), `VSpacing (0.5) ), `HSpacing (0.5))), - `VSpacing (), + `VSpacing () + /* FIXME: no import? `HBox ( // textentry label `TextEntry (`id(`import_dir), @@ -113,6 +116,7 @@ `PushButton (`id (`browse), Label::BrowseButton ()) ) ) + */ ), `HSpacing(3)); Wizard::SetContentsButtons (caption, con, help_text, @@ -123,26 +127,29 @@ if (!Stage::cont ()) Wizard::HideAbortButton (); - UI::ChangeWidget (`id (`import_dir), `Enabled, use_pam); - UI::ChangeWidget (`id (`browse), `Enabled, use_pam); +// UI::ChangeWidget (`id (`import_dir), `Enabled, use_pam); +// UI::ChangeWidget (`id (`browse), `Enabled, use_pam); any ret = nil; while (true) { ret = UI::UserInput(); - string import_dir = (string)UI::QueryWidget (`id (`import_dir), `Value); +// string import_dir = (string)UI::QueryWidget (`id (`import_dir), `Value); if (ret == `pamyes || ret == `pamno) { use_pam = (ret == `pamyes); + /* UI::ChangeWidget (`id (`import_dir), `Enabled, use_pam); UI::ChangeWidget (`id (`browse), `Enabled, use_pam); + */ } if (ret == `abort || ret == `cancel || ret == `back) { if(ReallyAbort()) break; else continue; } + /* else if (ret == `browse) { string dir = UI::AskForExistingDirectory (import_dir, ""); if (dir != nil) @@ -152,7 +159,9 @@ UI::ChangeWidget (`id(`import_dir), `Value, dir); } } + */ else if (ret == `next) { + /* if (use_pam && import_dir != "" && !FileUtils::IsDirectory (import_dir)) { @@ -162,6 +171,7 @@ UI::SetFocus (`id (`import_dir)); continue; } + */ if (use_pam && !Package::InstallAll (FingerprintReader::UpdatedArchPackages ( FingerprintReader::required_packages)) @@ -169,16 +179,17 @@ { use_pam = false; UI::ChangeWidget (`id (`rd), `Value, `pamno); + /* UI::ChangeWidget (`id (`import_dir), `Enabled, use_pam); UI::ChangeWidget (`id (`browse), `Enabled, use_pam); + */ continue; } - if (use_pam != FingerprintReader::use_pam || - (use_pam && import_dir != "")) + if (use_pam != FingerprintReader::use_pam) { FingerprintReader::modified = true; FingerprintReader::use_pam = use_pam; - FingerprintReader::import_dir = import_dir; +// FingerprintReader::import_dir = import_dir; } break; } Modified: trunk/fingerprint-reader/src/users_plugin_fingerprint_reader.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/fingerprint-reader/src/users_plu… ============================================================================== --- trunk/fingerprint-reader/src/users_plugin_fingerprint_reader.ycp (original) +++ trunk/fingerprint-reader/src/users_plugin_fingerprint_reader.ycp Mon Oct 6 16:24:06 2008 @@ -116,17 +116,6 @@ return ex; } - // helper function, form the string for UI from actuall data - void replace_swipe_string () { - - UI::ReplaceWidget (`id (`rpstatus), `Label ( - // status message (%1, %2 are numbers) - sformat (_("Successful swipes: %1, failed swipes: %2"), - swipe_success, swipe_failed) - )); - } - - void status_message (string label, string message) { UI::ReplaceWidget (`id (`rp), `Label (label)); UI::ReplaceWidget (`id (`rpstatus), `Label (message)); @@ -173,8 +162,6 @@ } case 3: { // FP_ENROLL_PASS swipe_success = swipe_success + 1; -// replace_swipe_string (); FIXME - status_message ( // status label _("Enroll stage passed."), @@ -294,7 +281,8 @@ { // modified data to add to user map<string, any> tmp_data = $[ - "_fingerprint_dir" : fingerprint_dir, // FIXME save correctly + // this is identifying the subdirectory of common tmp directory: + "_fingerprint_dir" : username, "plugin_modified" : 1 ]; if (data["what"]:"" == "edit_user") -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r51875 - /trunk/fingerprint-reader/yast2-fingerprint-reader.spec.in
by jsuchome＠svn.opensuse.org 06 Oct '08

06 Oct '08

Author: jsuchome Date: Mon Oct 6 16:10:59 2008 New Revision: 51875 URL: http://svn.opensuse.org/viewcvs/yast?rev=51875&view=rev Log: - removed Supplements from spec file (package will be recommended by pam_fp) Modified: trunk/fingerprint-reader/yast2-fingerprint-reader.spec.in Modified: trunk/fingerprint-reader/yast2-fingerprint-reader.spec.in URL: http://svn.opensuse.org/viewcvs/yast/trunk/fingerprint-reader/yast2-fingerp… ============================================================================== --- trunk/fingerprint-reader/yast2-fingerprint-reader.spec.in (original) +++ trunk/fingerprint-reader/yast2-fingerprint-reader.spec.in Mon Oct 6 16:10:59 2008 @@ -5,8 +5,6 @@ BuildRequires: libfprint0 libfprint-devel libusb-devel BuildRequires: doxygen gcc-c++ yast2-core-devel perl-XML-Writer update-desktop-files yast2 yast2-devtools yast2-testsuite yast2-pam -Supplements: modalias(usb:v0483p2016d*dc*dsc*dp*ic*isc*ip*) - Conflicts: yast2-hardware-detection < 2.15.7 Summary: Configuration of fingerprint reader -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0