Author: fehr Date: Wed Apr 11 16:26:06 2012 New Revision: 67913 URL: http://svn.opensuse.org/viewcvs/yast?rev=67913&view=rev Log: fix bug where ycp forgets changes in Storage::ChangeVolumeProperties after libstorage returns an error Modified: branches/SuSE-Code-11-SP3-Branch/storage/package/yast2-storage.changes branches/SuSE-Code-11-SP3-Branch/storage/storage/src/modules/Storage.ycp Modified: branches/SuSE-Code-11-SP3-Branch/storage/package/yast2-storage.changes URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP3-Branch/storage/package/yast2-storage.changes?rev=67913&r1=67912&r2=67913&view=diff ============================================================================== --- branches/SuSE-Code-11-SP3-Branch/storage/package/yast2-storage.changes (original) +++ branches/SuSE-Code-11-SP3-Branch/storage/package/yast2-storage.changes Wed Apr 11 16:26:06 2012 @@ -1,4 +1,10 @@ ------------------------------------------------------------------- +Wed Apr 11 16:23:09 CEST 2012 - fehr@suse.de + +- fix bug where ycp forgets changes in Storage::ChangeVolumeProperties + after libstorage returns an error + +------------------------------------------------------------------- Thu Apr 5 11:59:15 CEST 2012 - fehr@suse.de - suppress dm maps for "-real" and "-cow" of LVM snapshots in Modified: branches/SuSE-Code-11-SP3-Branch/storage/storage/src/modules/Storage.ycp URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP3-Branch/storage/storage/src/modules/Storage.ycp?rev=67913&r1=67912&r2=67913&view=diff ============================================================================== --- branches/SuSE-Code-11-SP3-Branch/storage/storage/src/modules/Storage.ycp (original) +++ branches/SuSE-Code-11-SP3-Branch/storage/storage/src/modules/Storage.ycp Wed Apr 11 16:26:06 2012 @@ -2748,12 +2748,14 @@ cre = remove( cre, 0 ); } } - if( ret==0 ) - { - y2debug( "ChangeVolumeProperties changed:%1 part:%2", changed, part ); - if( changed ) - UpdateTargetMapDev( dev ); - } + if( changed ) + UpdateTargetMapDev( dev ); + if( ret!=0 ) + { + y2milestone( "ChangeVolumeProperties ret:%1", ret ); + y2milestone( "ChangeVolumeProperties part:%1", part ); + y2milestone( "ChangeVolumeProperties curr:%1", curr ); + } return( ret==0 ); } -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org