On Tue, Jun 28, 2011 at 09:35:08AM -0000, [email protected] wrote: > URL: http://svn.opensuse.org/viewcvs/yast?rev=64609&view=rev > > Modified: trunk/add-on/src/add-on-workflow.ycp > URL: > http://svn.opensuse.org/viewcvs/yast/trunk/add-on/src/add-on-workflow.ycp?rev=64609&r1=64608&r2=64609&view=diff > ============================================================================== > --- trunk/add-on/src/add-on-workflow.ycp (original) > +++ trunk/add-on/src/add-on-workflow.ycp Tue Jun 28 11:35:07 2011 > @@ -180,6 +180,8 @@ > y2milestone ("Starting repository sequence"); > symbol ret = Sequencer::Run (aliases, sequence); > > + if (ret != `abort) > + { > list <integer> sources_after = Pkg::SourceGetCurrent (false); > y2milestone ("Sources with new one added: %1", sources_after); > > @@ -219,6 +221,7 @@ > y2milestone ("Saving all sources"); > Pkg::SourceSaveAll(); > } > + }
Ha! Bad indentation!
Please indent the code that is being put in a condition. If you
think that would be too much indentation, maybe it's a good
opportunity to factor it out.
>
> AddOnProduct::last_ret = ret;
> y2milestone ("MediaSelect Dialog ret: %1", ret);
--
Martin Vidner, YaST developer
http://en.opensuse.org/User:Mvidner
Kuracke oddeleni v restauraci je jako fekalni oddeleni v bazenu
pgpZU74K1Kn2h.pgp
Description: PGP signature
