Author: jsrain Date: Mon Nov 2 11:49:13 2009 New Revision: 59266 URL: http://svn.opensuse.org/viewcvs/yast?rev=59266&view=rev Log: warn in installation proposal if not formatting /home partition and using remote authentication (fate#306325) Modified: branches/SuSE-Code-11-SP1-Branch/storage/VERSION branches/SuSE-Code-11-SP1-Branch/storage/package/yast2-storage.changes branches/SuSE-Code-11-SP1-Branch/storage/storage/src/proposal/partitions_proposal.ycp branches/SuSE-Code-11-SP1-Branch/storage/yast2-storage.spec.in Modified: branches/SuSE-Code-11-SP1-Branch/storage/VERSION URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/storage/VERSION?rev=59266&r1=59265&r2=59266&view=diff ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/storage/VERSION (original) +++ branches/SuSE-Code-11-SP1-Branch/storage/VERSION Mon Nov 2 11:49:13 2009 @@ -1 +1 @@ -2.17.79 +2.17.80 Modified: branches/SuSE-Code-11-SP1-Branch/storage/package/yast2-storage.changes URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/storage/package/yast2-storage.changes?rev=59266&r1=59265&r2=59266&view=diff ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/storage/package/yast2-storage.changes (original) +++ branches/SuSE-Code-11-SP1-Branch/storage/package/yast2-storage.changes Mon Nov 2 11:49:13 2009 @@ -1,4 +1,11 @@ ------------------------------------------------------------------- +Mon Nov 2 11:25:05 CET 2009 - jsrain@suse.cz + +- warn in installation proposal if not formatting /home partition + and using remote authentication (fate#306325) +- 2.17.80 + +------------------------------------------------------------------- Wed Oct 14 16:55:15 CEST 2009 - aschnell@suse.de - do not detect MDs as disks (bnc #546912) Modified: branches/SuSE-Code-11-SP1-Branch/storage/storage/src/proposal/partitions_proposal.ycp URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/storage/storage/src/proposal/partitions_proposal.ycp?rev=59266&r1=59265&r2=59266&view=diff ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/storage/storage/src/proposal/partitions_proposal.ycp (original) +++ branches/SuSE-Code-11-SP1-Branch/storage/storage/src/proposal/partitions_proposal.ycp Mon Nov 2 11:49:13 2009 @@ -24,6 +24,25 @@ y2milestone( "func:%1 param:%2", func, param ); + // check if /home partition keeps unformatted in order to warn for + // possible incorrectly set file ownership (fate#306325) + boolean home_not_formatted() { + map tm = Storage::GetTargetMap(); + boolean ret = false; + foreach (string disk, map info, tm, { + list<map> partitions = info["partitions"]:[]; + foreach (map p, partitions, { + if (p["mount"]:"" == "/home" && ! p["format"]:false) + { + y2milestone ("The /home partition will not be formatted."); + ret = true; + break; + } + }); + }); + return ret; + } + if( func == "MakeProposal" && Mode::autoinst() ) { ret["preformatted_proposal"] = Storage::ChangeText(); @@ -64,6 +83,17 @@ !Storage::GetPartProposalActive() ) { ret["preformatted_proposal"] = Storage::ChangeText(); + // detect non-formated /home together with remote authentication - Fate#306325 + if (home_not_formatted()) + { + import "UsersSimple"; + if (UsersSimple::AfterAuth() != "users") + { + ret["warning"] = _("The /home partition will not be formatted. After installation, +ensure that the permissions to home directories are set properly."); + ret["warning_level"] = `warning; + } + } } else { Modified: branches/SuSE-Code-11-SP1-Branch/storage/yast2-storage.spec.in URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/storage/yast2-storage.spec.in?rev=59266&r1=59265&r2=59266&view=diff ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/storage/yast2-storage.spec.in (original) +++ branches/SuSE-Code-11-SP1-Branch/storage/yast2-storage.spec.in Mon Nov 2 11:49:13 2009 @@ -20,6 +20,7 @@ Obsoletes: yast2-trans-inst-partitioning Provides: y2t_inst-partitioning Obsoletes: y2t_inst-partitioning +Group: System/YaST Summary: - %description -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org