November 2008 - Zypp Commits - openSUSE Mailing Lists

[zypp-commit] r11824 - /trunk/libzypp/package/libzypp.changes
by mlandres＠svn.opensuse.org 28 Nov '08

28 Nov '08

Author: mlandres Date: Fri Nov 28 10:20:20 2008 New Revision: 11824 URL: http://svn.opensuse.org/viewcvs/zypp?rev=11824&view=rev Log: changes Modified: trunk/libzypp/package/libzypp.changes Modified: trunk/libzypp/package/libzypp.changes URL: http://svn.opensuse.org/viewcvs/zypp/trunk/libzypp/package/libzypp.changes?… ============================================================================== --- trunk/libzypp/package/libzypp.changes (original) +++ trunk/libzypp/package/libzypp.changes Fri Nov 28 10:20:20 2008 @@ -1,5 +1,5 @@ ------------------------------------------------------------------- -Fri Nov 28 00:05:17 CET 2008 - ma(a)calvin.km13.de +Fri Nov 28 00:05:17 CET 2008 - ma(a)suse.de - Take into account the requirements of all obsoleted packages uninstall scripts when computing the installation order. (bnc #439802) -- To unsubscribe, e-mail: zypp-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: zypp-commit+help(a)opensuse.org

1 0

[zypp-commit] r11823 - in /trunk/libzypp: VERSION.cmake devel/devel.ma/Iorder.cc package/libzypp.changes zypp/solver/detail/InstallOrder.cc
by mlandres＠svn.opensuse.org 27 Nov '08

27 Nov '08

Author: mlandres Date: Fri Nov 28 00:10:35 2008 New Revision: 11823 URL: http://svn.opensuse.org/viewcvs/zypp?rev=11823&view=rev Log: - Take into account the requirements of all obsoleted packages uninstall scripts when computing the installation order. (bnc #439802) Modified: trunk/libzypp/VERSION.cmake trunk/libzypp/devel/devel.ma/Iorder.cc trunk/libzypp/package/libzypp.changes trunk/libzypp/zypp/solver/detail/InstallOrder.cc Modified: trunk/libzypp/VERSION.cmake URL: http://svn.opensuse.org/viewcvs/zypp/trunk/libzypp/VERSION.cmake?rev=11823&… ============================================================================== --- trunk/libzypp/VERSION.cmake (original) +++ trunk/libzypp/VERSION.cmake Fri Nov 28 00:10:35 2008 @@ -61,8 +61,8 @@ SET(LIBZYPP_MAJOR "5") SET(LIBZYPP_COMPATMINOR "23") SET(LIBZYPP_MINOR "24") -SET(LIBZYPP_PATCH "2") +SET(LIBZYPP_PATCH "3") # -# LAST RELEASED: 5.24.2 (23) +# LAST RELEASED: 5.24.3 (23) # (The number in parenthesis is LIBZYPP_COMPATMINOR) #======= Modified: trunk/libzypp/devel/devel.ma/Iorder.cc URL: http://svn.opensuse.org/viewcvs/zypp/trunk/libzypp/devel/devel.ma/Iorder.cc… ============================================================================== --- trunk/libzypp/devel/devel.ma/Iorder.cc (original) +++ trunk/libzypp/devel/devel.ma/Iorder.cc Fri Nov 28 00:10:35 2008 @@ -1,48 +1,67 @@ #include "Tools.h" #include <zypp/ResObjects.h> +#include <zypp/ProgressData.h> +#include <zypp/sat/WhatObsoletes.h> #include "zypp/pool/GetResolvablesToInsDel.h" Pathname mroot( "/tmp/Bb" ); TestSetup test( mroot, Arch_ppc64 ); -bool checkCaps( Capabilities caps_r ); +#define LCStack "IOrder::Stack" +#define LCCache "IOrder::Cache" +#define LCVerbose "IOrder::Verbose" -struct RunableCache +bool progressReceiver( const ProgressData & v ) +{ + DBG << "...->" << v << endl; + return true; +} + +struct RunnableCache { typedef std::tr1::unordered_map<sat::Solvable,TriBool> CacheType; typedef std::vector<sat::Solvable> AnalyzeStack; - RunableCache() + RunnableCache() : _ltag( "[0000]" ) {} /** - * Test whether there is a runable provider for each requirement. + * Test whether there is a runnable provider for each requirement. */ - bool isRunable( const PoolItem & pi ) const - { return isRunable( pi.satSolvable() ); } + bool isRunnable( const PoolItem & pi ) const + { return isRunnable( pi.satSolvable() ); } - bool isRunable( sat::Solvable solv_r ) const + bool isRunnable( sat::Solvable solv_r ) const { - tribool & cent( get( solv_r ) ); - if ( indeterminate( cent ) ) - return (cent = analyze( solv_r )); - return cent; + SEC << "Runnable? " << solv_r << endl; + if ( _isRunnable( solv_r ) ) + { + MIL << "Runnable: " << solv_r << endl; + return true; + } + ERR << "NotRunnable: " << solv_r << endl; + return false; } /** - * Test whether there is a runable provider for each pre-requirement. + * Test whether there is a runnable provider for each pre-requirement. */ bool isInstallable( const PoolItem & pi ) const { return isInstallable( pi.satSolvable() ); } bool isInstallable( sat::Solvable solv_r ) const { - tribool & cent( get( solv_r ) ); - if ( cent ) - return true; // if runable then also installable. - return checkCaps( solv_r.prerequires() ); + SEC << "Installable? " << solv_r << endl; + tribool & cent( get( solv_r ) ); // if (cached) runnable then also installable. + if ( cent || checkCaps( solv_r.prerequires() ) ) + { + MIL << "Installable: " << solv_r << endl; + return true; + } + ERR << "NotInstallable: " << solv_r << endl; + return false; } /** Clear the cache. */ @@ -51,83 +70,90 @@ _cache.clear(); _stack.clear(); _ltag = "[0000]"; + _INT(LCCache) << "Cache cleared!" << endl; } private: + /** Internal version without loging for recursive calls. */ + bool _isRunnable( const PoolItem & pi ) const + { return _isRunnable( pi.satSolvable() ); } + + bool _isRunnable( sat::Solvable solv_r ) const + { + tribool & cent( get( solv_r ) ); + if ( indeterminate( cent ) ) + cent = analyze( solv_r ); + return cent; + } + /** - * Determine whether this solvable is runable. + * Determine whether this solvable is runnable. */ bool analyze( sat::Solvable solv_r ) const { if ( ! push( solv_r ) ) { if ( _stack.back() != solv_r ) - SEC << _ltag << "??Already on stack: " << solv_r << " " << _stack << endl; + _SEC(LCStack) << _ltag << "** CYCLE: " << solv_r << " " << _stack << endl; // else it's a self requirement return true; // assume runnable? } - - INT << _ltag << "->" << solv_r << " " << _stack << endl; + _INT(LCStack) << _ltag << "->" << solv_r << " " << _stack << endl; bool ret = checkCaps( solv_r.requires() ); - if ( ! ret ) - { - ERR << " Not runable: " << solv_r << endl; - } - INT << _ltag << "<-" << solv_r << " " << _stack << endl; - + _INT(LCStack) << _ltag << "<-" << solv_r << " " << _stack << endl; if ( ! pop( solv_r ) ) { - INT << "Stack corrupted! Expect " << solv_r << " " << _stack << endl; + _SEC(LCStack) << "** Stack corrupted! Expect " << solv_r << " " << _stack << endl; } return ret; } /** - * For each capability find a runable provider. + * For each capability find a runnable provider. */ bool checkCaps( Capabilities caps_r ) const { for_( it, caps_r.begin(), caps_r.end() ) { - if ( ! findRunableProvider( *it ) ) + if ( ! findRunnableProvider( *it ) ) return false; } return true; } /** - * Find a runable provider of a capability on system. + * Find a runnable provider of a capability on system. * - * A runable package is already installed and all of - * its requirements are met by runable packages. + * A runnable package is already installed and all of + * its requirements are met by runnable packages. */ - bool findRunableProvider( Capability cap_r ) const + bool findRunnableProvider( Capability cap_r ) const { - _MIL("findRunableProvider") << _ltag << " " << cap_r << endl; + _MIL(LCVerbose) << _ltag << " " << cap_r << endl; sat::WhatProvides prv( cap_r ); for_( pit, prv.begin(), prv.end() ) { if ( ! *pit ) { - _DBG("findRunableProvider") << _ltag << " by system" << endl; + _DBG(LCVerbose) << _ltag << " by system" << endl; return true; // noSolvable provides: i.e. system provides } PoolItem pi( *pit ); if ( pi.status().onSystem() ) { - if ( isRunable( pi ) ) + if ( _isRunnable( pi ) ) { - _DBG("findRunableProvider") << _ltag << " " << pi << endl; + _DBG(LCVerbose) << _ltag << " " << pi << endl; return true; } else { - _WAR("findRunableProvider") << _ltag << " " << pi << endl; + _WAR(LCVerbose) << _ltag << " " << pi << endl; } } } - ERR << _ltag << " NO on system provider for " << cap_r << endl; + ERR << _ltag << " NO runnable provider for " << cap_r << endl; return false; } @@ -138,7 +164,7 @@ if ( find( _stack.begin(), _stack.end(), solv_r ) == _stack.end() ) { _stack.push_back( solv_r ); - _ltag = str::form( "[%04u]", _stack.size() ); + _ltag = str::form( "[%04lu]", _stack.size() ); return true; } // cycle? @@ -151,7 +177,7 @@ if ( _stack.back() == solv_r ) { _stack.pop_back(); - _ltag = str::form( "[%04u]", _stack.size() ); + _ltag = str::form( "[%0l4u]", _stack.size() ); return true; } // stack corrupted? @@ -172,7 +198,7 @@ mutable std::string _ltag; }; -RunableCache rcache; +RunnableCache rcache; //================================================== @@ -239,6 +265,104 @@ test.poolProxy().restoreState(); } +void display( const pool::GetResolvablesToInsDel & collect, std::set<IdString> interested ) +{ + USR << "======================================================================" << endl; + USR << "=== DELETE" << endl; + USR << "======================================================================" << endl; + if ( 1 ) + { + ProgressData tics( collect._toDelete.size() ); + tics.name( "DELETE" ); + tics.sendTo( &progressReceiver ); + tics.toMin(); + + for_( it, collect._toDelete.begin(), collect._toDelete.end() ) + { + tics.incr(); + + it->status().setTransact( true, ResStatus::USER ); +// vdumpPoolStats( SEC << "Transacting:"<< endl, +// make_filter_begin<resfilter::ByTransact>(pool), +// make_filter_end<resfilter::ByTransact>(pool) ) << endl; + + if ( !interested.empty() && interested.find( it->satSolvable().ident() ) == interested.end() ) + { + MIL << "..." << *it << endl; + continue; + } + + rcache.clear(); + if ( ! rcache.isInstallable( *it ) ) + { + USR << "FAILED DEL " << *it << endl; + } + } + } + + USR << "======================================================================" << endl; + USR << "=== INSTALL" << endl; + USR << "======================================================================" << endl; + if ( 1 ) + { + ProgressData tics( collect._toInstall.size() ); + tics.name( "INSTALL" ); + tics.sendTo( progressReceiver ); + tics.toMin(); + + + for_( it, collect._toInstall.begin(), collect._toInstall.end() ) + { + tics.incr(); + + ui::Selectable::Ptr p( ui::Selectable::get( *it ) ); + p->setCandidate( *it ); + p->setToInstall(); // also deletes the installed one +// vdumpPoolStats( SEC << "Transacting:"<< endl, +// make_filter_begin<resfilter::ByTransact>(pool), +// make_filter_end<resfilter::ByTransact>(pool) ) << endl; + + + if ( !interested.empty() && interested.find( p->ident() ) == interested.end() ) + { + MIL << "..." << *it << endl; + continue; + } + + rcache.clear(); + + sat::WhatObsoletes obs( *it ); + for_( it, obs.begin(), obs.end() ) + { + if ( ! rcache.isInstallable( *it ) ) + { + USR << "FAILED OBS " << *it << endl; + } + } + + if ( ! rcache.isInstallable( *it ) ) + { + USR << "FAILED INS " << *it << endl; + } + } + } +} + +void display( const pool::GetResolvablesToInsDel & collect, IdString ident_r ) +{ + std::set<IdString> interested; + interested.insert( ident_r ); + display( collect, interested ); +} + + +void display( const pool::GetResolvablesToInsDel & collect ) +{ + std::set<IdString> interested; + display( collect, interested ); +} + + /****************************************************************** ** ** FUNCTION NAME : main @@ -262,8 +386,8 @@ if ( 0 ) { - PoolItem p = getPi<Package>( "bash", Edition("3.1-24.14"), Arch("ppc64") ); - p.status().setTransact( true, ResStatus::USER ); + PoolItem pi = getPi<Package>( "bash", Edition("3.1-24.14"), Arch("ppc64") ); + pi.status().setTransact( true, ResStatus::USER ); } save(); @@ -278,97 +402,19 @@ make_filter_end<resfilter::ByTransact>(pool) ) << endl; pool::GetResolvablesToInsDel collect( pool, pool::GetResolvablesToInsDel::ORDER_BY_MEDIANR ); - restore(); - vdumpPoolStats( USR << "Transacting:"<< endl, - make_filter_begin<resfilter::ByTransact>(pool), - make_filter_end<resfilter::ByTransact>(pool) ) << endl; - - { - for_( it, collect._toDelete.begin(), collect._toDelete.end() ) - { - it->status().setTransact( true, ResStatus::USER ); - SEC << *it << endl; -// vdumpPoolStats( SEC << "Transacting:"<< endl, -// make_filter_begin<resfilter::ByTransact>(pool), -// make_filter_end<resfilter::ByTransact>(pool) ) << endl; - - (rcache.isInstallable( *it ) ? MIL : ERR) << *it << " deletable?" << endl; - } - } - - if ( 0 ) { - for_( it, collect._toInstall.begin(), collect._toInstall.end() ) - { - it->status().setTransact( true, ResStatus::USER ); - SEC << *it << endl; -// vdumpPoolStats( SEC << "Transacting:"<< endl, -// make_filter_begin<resfilter::ByTransact>(pool), -// make_filter_end<resfilter::ByTransact>(pool) ) << endl; - (rcache.isInstallable( *it ) ? MIL : ERR) << *it << " installable?" << endl; - } - } - - -#if 0 - //getPi<>( "", "", Edition(""), Arch("") ); - getPi<Product>( "SUSE_SLES", Edition("11"), Arch("ppc64") ).status().setTransact( true, ResStatus::USER ); - getPi<Package>( "sles-release", Edition("11-54.3"), Arch("ppc64") ).status().setTransact( true, ResStatus::USER ); - - ResPool pool( test.pool() ); - vdumpPoolStats( USR << "Transacting:"<< endl, - make_filter_begin<resfilter::ByTransact>(pool), - make_filter_end<resfilter::ByTransact>(pool) ) << endl; - upgrade(); - vdumpPoolStats( USR << "Transacting:"<< endl, - make_filter_begin<resfilter::ByTransact>(pool), - make_filter_end<resfilter::ByTransact>(pool) ) << endl; - - pool::GetResolvablesToInsDel collect( pool, pool::GetResolvablesToInsDel::ORDER_BY_MEDIANR ); - MIL << "GetResolvablesToInsDel:" << endl << collect << endl; - - if ( 1 ) - { - // Collect until the 1st package from an unwanted media occurs. - // Further collection could violate install order. - bool hitUnwantedMedia = false; - typedef pool::GetResolvablesToInsDel::PoolItemList PoolItemList; - PoolItemList::iterator fst=collect._toInstall.end(); - for ( PoolItemList::iterator it = collect._toInstall.begin(); it != collect._toInstall.end(); ++it) - { - ResObject::constPtr res( it->resolvable() ); - - if ( hitUnwantedMedia - || ( res->mediaNr() && res->mediaNr() != 1 ) ) - { - if ( !hitUnwantedMedia ) - fst=it; - hitUnwantedMedia = true; - } - } - dumpRange( WAR << "toInstall1: " << endl, - collect._toInstall.begin(), fst ) << endl; - dumpRange( WAR << "toInstall2: " << endl, - fst, collect._toInstall.end() ) << endl; - dumpRange( ERR << "toDelete: " << endl, - collect._toDelete.begin(), collect._toDelete.end() ) << endl; - } - else - { - dumpRange( WAR << "toInstall: " << endl, - collect._toInstall.begin(), collect._toInstall.end() ) << endl; - dumpRange( ERR << "toDelete: " << endl, - collect._toDelete.begin(), collect._toDelete.end() ) << endl; - } - INT << "===[END]============================================" << endl << endl; - return 0; -#endif + USR << ui::Selectable::get( "libcdio" ) << endl; + USR << ui::Selectable::get( "libcdio7" ) << endl; + std::set<IdString> interested; + //interested.insert( IdString("libcdio") ); + //interested.insert( IdString("libcdio7") ); + restore(); + display( collect, interested ); - INT << "===[END]============================================" << endl << endl; + INT << "===[END]============================================" << endl << endl; zypp::base::LogControl::TmpLineWriter shutUp; return 0; } - Modified: trunk/libzypp/package/libzypp.changes URL: http://svn.opensuse.org/viewcvs/zypp/trunk/libzypp/package/libzypp.changes?… ============================================================================== --- trunk/libzypp/package/libzypp.changes (original) +++ trunk/libzypp/package/libzypp.changes Fri Nov 28 00:10:35 2008 @@ -1,4 +1,12 @@ ------------------------------------------------------------------- +Fri Nov 28 00:05:17 CET 2008 - ma(a)calvin.km13.de + +- Take into account the requirements of all obsoleted packages uninstall + scripts when computing the installation order. (bnc #439802) +- revision 11823 +- version 5.24.3 (23) + +------------------------------------------------------------------- Thu Nov 27 14:45:07 CET 2008 - dmacvicar(a)suse.de - fix maybeUnsuported() method returning wrong result Modified: trunk/libzypp/zypp/solver/detail/InstallOrder.cc URL: http://svn.opensuse.org/viewcvs/zypp/trunk/libzypp/zypp/solver/detail/Insta… ============================================================================== --- trunk/libzypp/zypp/solver/detail/InstallOrder.cc (original) +++ trunk/libzypp/zypp/solver/detail/InstallOrder.cc Fri Nov 28 00:10:35 2008 @@ -38,6 +38,7 @@ #include "zypp/ResStatus.h" #include "zypp/NameKindProxy.h" #include "zypp/sat/Pool.h" +#include <zypp/sat/WhatObsoletes.h> ///////////////////////////////////////////////////////////////////////// namespace zypp @@ -196,12 +197,12 @@ // items prereq CapabilitySet prq( item->dep(Dep::PREREQUIRES).begin(), item->dep(Dep::PREREQUIRES).end() ); - // an installed items prereq (in case they are reqired for uninstall scripts) - NameKindProxy nkp( _pool, item->name(), item->kind() ); - if ( ! nkp.installedEmpty() ) + // any obsoleted items prereq (in case they are reqired for uninstall scripts) + sat::WhatObsoletes obs( item ); + for_( it, obs.begin(), obs.end() ) { - prq.insert( (*nkp.installedBegin())->dep(Dep::PREREQUIRES).begin(), - (*nkp.installedBegin())->dep(Dep::PREREQUIRES).end() ); + Capabilities p( it->prerequires() ); + prq.insert( p.begin(), p.end() ); } // put prerequires first and requires last on list to ensure // that prerequires are processed first -- To unsubscribe, e-mail: zypp-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: zypp-commit+help(a)opensuse.org

1 0

[zypp-commit] r11822 - /trunk/libzypp/VERSION.cmake
by dmacvicar＠svn.opensuse.org 27 Nov '08

27 Nov '08

Author: dmacvicar Date: Thu Nov 27 15:26:28 2008 New Revision: 11822 URL: http://svn.opensuse.org/viewcvs/zypp?rev=11822&view=rev Log: bump Modified: trunk/libzypp/VERSION.cmake Modified: trunk/libzypp/VERSION.cmake URL: http://svn.opensuse.org/viewcvs/zypp/trunk/libzypp/VERSION.cmake?rev=11822&… ============================================================================== --- trunk/libzypp/VERSION.cmake (original) +++ trunk/libzypp/VERSION.cmake Thu Nov 27 15:26:28 2008 @@ -63,6 +63,6 @@ SET(LIBZYPP_MINOR "24") SET(LIBZYPP_PATCH "2") # -# LAST RELEASED: 5.24.1 (23) +# LAST RELEASED: 5.24.2 (23) # (The number in parenthesis is LIBZYPP_COMPATMINOR) #======= -- To unsubscribe, e-mail: zypp-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: zypp-commit+help(a)opensuse.org

1 0

[zypp-commit] r11821 - /trunk/libzypp/package/libzypp.changes
by dmacvicar＠svn.opensuse.org 27 Nov '08

27 Nov '08

Author: dmacvicar Date: Thu Nov 27 15:26:27 2008 New Revision: 11821 URL: http://svn.opensuse.org/viewcvs/zypp?rev=11821&view=rev Log: changes and release Modified: trunk/libzypp/package/libzypp.changes Modified: trunk/libzypp/package/libzypp.changes URL: http://svn.opensuse.org/viewcvs/zypp/trunk/libzypp/package/libzypp.changes?… ============================================================================== --- trunk/libzypp/package/libzypp.changes (original) +++ trunk/libzypp/package/libzypp.changes Thu Nov 27 15:26:27 2008 @@ -4,6 +4,7 @@ - fix maybeUnsuported() method returning wrong result - add testcase for future coverage - don't force time based uuid for anonymous id string (bnc #449654) +- RELEASE: 5.24.2 (23) ------------------------------------------------------------------- Wed Nov 26 17:18:00 CET 2008 - ma(a)suse.de -- To unsubscribe, e-mail: zypp-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: zypp-commit+help(a)opensuse.org

1 0

[zypp-commit] r11820 - /trunk/libzypp/package/libzypp.changes
by dmacvicar＠svn.opensuse.org 27 Nov '08

27 Nov '08

Author: dmacvicar Date: Thu Nov 27 15:12:53 2008 New Revision: 11820 URL: http://svn.opensuse.org/viewcvs/zypp?rev=11820&view=rev Log: append changes Modified: trunk/libzypp/package/libzypp.changes Modified: trunk/libzypp/package/libzypp.changes URL: http://svn.opensuse.org/viewcvs/zypp/trunk/libzypp/package/libzypp.changes?… ============================================================================== --- trunk/libzypp/package/libzypp.changes (original) +++ trunk/libzypp/package/libzypp.changes Thu Nov 27 15:12:53 2008 @@ -3,6 +3,7 @@ - fix maybeUnsuported() method returning wrong result - add testcase for future coverage +- don't force time based uuid for anonymous id string (bnc #449654) ------------------------------------------------------------------- Wed Nov 26 17:18:00 CET 2008 - ma(a)suse.de -- To unsubscribe, e-mail: zypp-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: zypp-commit+help(a)opensuse.org

1 0

[zypp-commit] r11819 - /trunk/libzypp/zypp/target/TargetImpl.cc
by dmacvicar＠svn.opensuse.org 27 Nov '08

27 Nov '08

Author: dmacvicar Date: Thu Nov 27 15:12:52 2008 New Revision: 11819 URL: http://svn.opensuse.org/viewcvs/zypp?rev=11819&view=rev Log: - don't force time based uuid for anonymous id string (bnc #449654) Modified: trunk/libzypp/zypp/target/TargetImpl.cc Modified: trunk/libzypp/zypp/target/TargetImpl.cc URL: http://svn.opensuse.org/viewcvs/zypp/trunk/libzypp/zypp/target/TargetImpl.c… ============================================================================== --- trunk/libzypp/zypp/target/TargetImpl.cc (original) +++ trunk/libzypp/zypp/target/TargetImpl.cc Thu Nov 27 15:12:52 2008 @@ -312,8 +312,6 @@ const char* argv[] = { "/usr/bin/uuidgen", - "-r", - "-t", NULL }; -- To unsubscribe, e-mail: zypp-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: zypp-commit+help(a)opensuse.org

1 0

[zypp-commit] r11818 - /trunk/libzypp/package/libzypp.changes
by dmacvicar＠svn.opensuse.org 27 Nov '08

27 Nov '08

Author: dmacvicar Date: Thu Nov 27 14:45:39 2008 New Revision: 11818 URL: http://svn.opensuse.org/viewcvs/zypp?rev=11818&view=rev Log: changes Modified: trunk/libzypp/package/libzypp.changes Modified: trunk/libzypp/package/libzypp.changes URL: http://svn.opensuse.org/viewcvs/zypp/trunk/libzypp/package/libzypp.changes?… ============================================================================== --- trunk/libzypp/package/libzypp.changes (original) +++ trunk/libzypp/package/libzypp.changes Thu Nov 27 14:45:39 2008 @@ -1,4 +1,10 @@ ------------------------------------------------------------------- +Thu Nov 27 14:45:07 CET 2008 - dmacvicar(a)suse.de + +- fix maybeUnsuported() method returning wrong result +- add testcase for future coverage + +------------------------------------------------------------------- Wed Nov 26 17:18:00 CET 2008 - ma(a)suse.de - Call 'repo2solv -R' (recursive scan) for plaindir repos. (bnc #443350) -- To unsubscribe, e-mail: zypp-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: zypp-commit+help(a)opensuse.org

1 0

[zypp-commit] r11817 - in /trunk/libzypp: tests/repo/ tests/repo/yum/data/extensions/repodata/ zypp/
by dmacvicar＠svn.opensuse.org 27 Nov '08

27 Nov '08

Author: dmacvicar Date: Thu Nov 27 14:45:37 2008 New Revision: 11817 URL: http://svn.opensuse.org/viewcvs/zypp?rev=11817&view=rev Log: - fix maybeUnsuported() method returning wrong result - add testcase for future coverage Added: trunk/libzypp/tests/repo/yum/data/extensions/repodata/susedata.xml.gz Modified: trunk/libzypp/tests/repo/ExtendedMetadata_test.cc trunk/libzypp/tests/repo/yum/data/extensions/repodata/filelists.xml.gz trunk/libzypp/tests/repo/yum/data/extensions/repodata/other.xml.gz trunk/libzypp/tests/repo/yum/data/extensions/repodata/primary.xml.gz trunk/libzypp/tests/repo/yum/data/extensions/repodata/repomd.xml trunk/libzypp/zypp/Package.cc Modified: trunk/libzypp/tests/repo/ExtendedMetadata_test.cc URL: http://svn.opensuse.org/viewcvs/zypp/trunk/libzypp/tests/repo/ExtendedMetad… ============================================================================== --- trunk/libzypp/tests/repo/ExtendedMetadata_test.cc (original) +++ trunk/libzypp/tests/repo/ExtendedMetadata_test.cc Thu Nov 27 14:45:37 2008 @@ -10,6 +10,7 @@ #include "zypp/base/Exception.h" #include "zypp/PathInfo.h" #include "zypp/TmpPath.h" +#include "zypp/Package.h" #include "zypp/RepoManager.h" #include "zypp/sat/Pool.h" #include "KeyRingTestReceiver.h" @@ -91,4 +92,53 @@ BOOST_CHECK( repo.providesUpdatesFor("cpe://o:sle") ); BOOST_CHECK( ! repo.providesUpdatesFor("cpe://o:windows") ); + // reuse to count solvables + count = 0; + + /** + * Now check for the extended metadata of the packages + */ + for_( it, repo.solvablesBegin(), repo.solvablesEnd() ) + { + sat::Solvable s = *it; + MIL << s << endl; + MIL << s.kind() << endl; + if ( s.ident() == "wt" ) + { + count++; + Package::Ptr p = asKind<Package>(makeResObject(s)); + BOOST_CHECK(p); + BOOST_CHECK(p->maybeUnsupported() ); + BOOST_CHECK_EQUAL(p->vendorSupport(), VendorSupportUnknown ); + + } + else if ( s.ident() == "foobar" ) + { + count++; + Package::Ptr p = asKind<Package>(makeResObject(s)); + BOOST_CHECK(p); + BOOST_CHECK_EQUAL(p->vendorSupport(), VendorSupportUnsupported ); + BOOST_CHECK(p->maybeUnsupported() ); + } + else if ( s.ident() == "foofoo" ) + { + count++; + Package::Ptr p = asKind<Package>(makeResObject(s)); + BOOST_CHECK(p); + // if it is level 3 support it cant be unsupported + BOOST_CHECK_EQUAL(p->vendorSupport(), VendorSupportLevel3 ); + BOOST_CHECK(! p->maybeUnsupported() ); + + } + else + { + BOOST_FAIL(str::form("Repo has package not contemplated in test: %s", s.ident().c_str()).c_str()); + } + + } + + // check that we actually found all testeable + // resolvables + BOOST_CHECK_EQUAL(count, 3); + } Modified: trunk/libzypp/tests/repo/yum/data/extensions/repodata/filelists.xml.gz URL: http://svn.opensuse.org/viewcvs/zypp/trunk/libzypp/tests/repo/yum/data/exte… ============================================================================== Files trunk/libzypp/tests/repo/yum/data/extensions/repodata/filelists.xml.gz (original) and trunk/libzypp/tests/repo/yum/data/extensions/repodata/filelists.xml.gz Thu Nov 27 14:45:37 2008 differ Modified: trunk/libzypp/tests/repo/yum/data/extensions/repodata/other.xml.gz URL: http://svn.opensuse.org/viewcvs/zypp/trunk/libzypp/tests/repo/yum/data/exte… ============================================================================== Files trunk/libzypp/tests/repo/yum/data/extensions/repodata/other.xml.gz (original) and trunk/libzypp/tests/repo/yum/data/extensions/repodata/other.xml.gz Thu Nov 27 14:45:37 2008 differ Modified: trunk/libzypp/tests/repo/yum/data/extensions/repodata/primary.xml.gz URL: http://svn.opensuse.org/viewcvs/zypp/trunk/libzypp/tests/repo/yum/data/exte… ============================================================================== Files trunk/libzypp/tests/repo/yum/data/extensions/repodata/primary.xml.gz (original) and trunk/libzypp/tests/repo/yum/data/extensions/repodata/primary.xml.gz Thu Nov 27 14:45:37 2008 differ Modified: trunk/libzypp/tests/repo/yum/data/extensions/repodata/repomd.xml URL: http://svn.opensuse.org/viewcvs/zypp/trunk/libzypp/tests/repo/yum/data/exte… ============================================================================== --- trunk/libzypp/tests/repo/yum/data/extensions/repodata/repomd.xml (original) +++ trunk/libzypp/tests/repo/yum/data/extensions/repodata/repomd.xml Thu Nov 27 14:45:37 2008 @@ -2,23 +2,29 @@ <repomd xmlns="http://linux.duke.edu/metadata/repo"> <data type="other"> <location href="repodata/other.xml.gz"/> - <checksum type="sha">73917678784071a9e1f66d333a3a265d01469098</checksum> + <checksum type="sha">603807e12e7418911fa9f158ef66b9c20f1df567</checksum> <timestamp>1215823452</timestamp> <open-checksum type="sha">34cde52910a161664291bd93e8a9cd24f751af6b</open-checksum> </data> <data type="filelists"> <location href="repodata/filelists.xml.gz"/> - <checksum type="sha">5df72494136a4e4b608ce3119802d6aa8d0f1167</checksum> + <checksum type="sha">06660c856facf17822102d09d849d42fad79d28e</checksum> <timestamp>1215823453</timestamp> <open-checksum type="sha">fdabcfd9c056109ab25acf0644823bdec94b6cfd</open-checksum> </data> <data type="primary"> <location href="repodata/primary.xml.gz"/> - <checksum type="sha">52969c92f9c595108376cd37df7d5ada41851f6a</checksum> + <checksum type="sha">2eccb910f933ee28fd5f5223e05eab6c5b08db67</checksum> <timestamp>1215823454</timestamp> <open-checksum type="sha">c6bd3c6099de74d97ed045163235be10ffa8a85f</open-checksum> </data> + <data type="susedata"> + <location href="repodata/susedata.xml.gz"/> + <checksum type="sha">359396be40992603aecf6e2832839111e09eb080</checksum> + <timestamp>1227279057.0</timestamp> + <open-checksum type="sha">b17c055bef95bca397faffdf028cfa91dd1b24bc</open-checksum> + </data> <data type="suseinfo"> <location href="repodata/suseinfo.xml.gz"/> Added: trunk/libzypp/tests/repo/yum/data/extensions/repodata/susedata.xml.gz URL: http://svn.opensuse.org/viewcvs/zypp/trunk/libzypp/tests/repo/yum/data/exte… ============================================================================== Files trunk/libzypp/tests/repo/yum/data/extensions/repodata/susedata.xml.gz (added) and trunk/libzypp/tests/repo/yum/data/extensions/repodata/susedata.xml.gz Thu Nov 27 14:45:37 2008 differ Modified: trunk/libzypp/zypp/Package.cc URL: http://svn.opensuse.org/viewcvs/zypp/trunk/libzypp/zypp/Package.cc?rev=1181… ============================================================================== --- trunk/libzypp/zypp/Package.cc (original) +++ trunk/libzypp/zypp/Package.cc Thu Nov 27 14:45:37 2008 @@ -65,7 +65,11 @@ bool Package::maybeUnsupported() const { - return ( vendorSupport() & ( VendorSupportACC | VendorSupportUnsupported | VendorSupportUnknown ) ); + if ( ( vendorSupport() == VendorSupportUnknown ) || + ( vendorSupport() == VendorSupportACC ) || + ( vendorSupport() == VendorSupportUnsupported ) ) + return true; + return false; } Changelog Package::changelog() const -- To unsubscribe, e-mail: zypp-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: zypp-commit+help(a)opensuse.org

1 0

[zypp-commit] r11816 - /trunk/libzypp/zypp/ProgressData.cc
by mlandres＠svn.opensuse.org 27 Nov '08

27 Nov '08

Author: mlandres Date: Thu Nov 27 14:42:09 2008 New Revision: 11816 URL: http://svn.opensuse.org/viewcvs/zypp?rev=11816&view=rev Log: fix debug output Modified: trunk/libzypp/zypp/ProgressData.cc Modified: trunk/libzypp/zypp/ProgressData.cc URL: http://svn.opensuse.org/viewcvs/zypp/trunk/libzypp/zypp/ProgressData.cc?rev… ============================================================================== --- trunk/libzypp/zypp/ProgressData.cc (original) +++ trunk/libzypp/zypp/ProgressData.cc Thu Nov 27 14:42:09 2008 @@ -108,9 +108,9 @@ { if ( obj.hasRange() ) { - return str << str::form( "{%u|%s}[%lld,%lld](%lld)", + return str << str::form( "{%u|%s}[%lld,%lld](%lld)%lld%%)", obj.numericId(), obj.name().c_str(), - obj.min(), obj.max(), obj.val() ); + obj.min(), obj.max(), obj.val(), obj.reportValue() ); } return str << str::form( "{%u|%s}[-,-](%lld)", obj.numericId(), obj.name().c_str(), -- To unsubscribe, e-mail: zypp-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: zypp-commit+help(a)opensuse.org

1 0

[zypp-commit] r11815 - /trunk/libzypp/devel/devel.ma/Iorder.cc
by mlandres＠svn.opensuse.org 26 Nov '08

26 Nov '08

Author: mlandres Date: Wed Nov 26 18:32:05 2008 New Revision: 11815 URL: http://svn.opensuse.org/viewcvs/zypp?rev=11815&view=rev Log: backup Modified: trunk/libzypp/devel/devel.ma/Iorder.cc Modified: trunk/libzypp/devel/devel.ma/Iorder.cc URL: http://svn.opensuse.org/viewcvs/zypp/trunk/libzypp/devel/devel.ma/Iorder.cc… ============================================================================== --- trunk/libzypp/devel/devel.ma/Iorder.cc (original) +++ trunk/libzypp/devel/devel.ma/Iorder.cc Wed Nov 26 18:32:05 2008 @@ -6,6 +6,176 @@ Pathname mroot( "/tmp/Bb" ); TestSetup test( mroot, Arch_ppc64 ); +bool checkCaps( Capabilities caps_r ); + +struct RunableCache +{ + typedef std::tr1::unordered_map<sat::Solvable,TriBool> CacheType; + typedef std::vector<sat::Solvable> AnalyzeStack; + + RunableCache() + : _ltag( "[0000]" ) + {} + + /** + * Test whether there is a runable provider for each requirement. + */ + bool isRunable( const PoolItem & pi ) const + { return isRunable( pi.satSolvable() ); } + + bool isRunable( sat::Solvable solv_r ) const + { + tribool & cent( get( solv_r ) ); + if ( indeterminate( cent ) ) + return (cent = analyze( solv_r )); + return cent; + } + + /** + * Test whether there is a runable provider for each pre-requirement. + */ + bool isInstallable( const PoolItem & pi ) const + { return isInstallable( pi.satSolvable() ); } + + bool isInstallable( sat::Solvable solv_r ) const + { + tribool & cent( get( solv_r ) ); + if ( cent ) + return true; // if runable then also installable. + return checkCaps( solv_r.prerequires() ); + } + + /** Clear the cache. */ + void clear() const + { + _cache.clear(); + _stack.clear(); + _ltag = "[0000]"; + } + + private: + /** + * Determine whether this solvable is runable. + */ + bool analyze( sat::Solvable solv_r ) const + { + if ( ! push( solv_r ) ) + { + if ( _stack.back() != solv_r ) + SEC << _ltag << "??Already on stack: " << solv_r << " " << _stack << endl; + // else it's a self requirement + return true; // assume runnable? + } + + INT << _ltag << "->" << solv_r << " " << _stack << endl; + bool ret = checkCaps( solv_r.requires() ); + if ( ! ret ) + { + ERR << " Not runable: " << solv_r << endl; + } + INT << _ltag << "<-" << solv_r << " " << _stack << endl; + + if ( ! pop( solv_r ) ) + { + INT << "Stack corrupted! Expect " << solv_r << " " << _stack << endl; + } + return ret; + } + + /** + * For each capability find a runable provider. + */ + bool checkCaps( Capabilities caps_r ) const + { + for_( it, caps_r.begin(), caps_r.end() ) + { + if ( ! findRunableProvider( *it ) ) + return false; + } + return true; + } + + /** + * Find a runable provider of a capability on system. + * + * A runable package is already installed and all of + * its requirements are met by runable packages. + */ + bool findRunableProvider( Capability cap_r ) const + { + _MIL("findRunableProvider") << _ltag << " " << cap_r << endl; + sat::WhatProvides prv( cap_r ); + for_( pit, prv.begin(), prv.end() ) + { + if ( ! *pit ) + { + _DBG("findRunableProvider") << _ltag << " by system" << endl; + return true; // noSolvable provides: i.e. system provides + } + + PoolItem pi( *pit ); + if ( pi.status().onSystem() ) + { + if ( isRunable( pi ) ) + { + _DBG("findRunableProvider") << _ltag << " " << pi << endl; + return true; + } + else + { + _WAR("findRunableProvider") << _ltag << " " << pi << endl; + } + } + } + ERR << _ltag << " NO on system provider for " << cap_r << endl; + return false; + } + + private: + /** Push a new solvable to the AnalyzeStack, or return false is already on stack. */ + bool push( sat::Solvable solv_r ) const + { + if ( find( _stack.begin(), _stack.end(), solv_r ) == _stack.end() ) + { + _stack.push_back( solv_r ); + _ltag = str::form( "[%04u]", _stack.size() ); + return true; + } + // cycle? + return false; + } + + /** Pop solvable from AnalyzeStack (expecting it to be \c solv_r). */ + bool pop( sat::Solvable solv_r ) const + { + if ( _stack.back() == solv_r ) + { + _stack.pop_back(); + _ltag = str::form( "[%04u]", _stack.size() ); + return true; + } + // stack corrupted? + return false; + } + + /** Return cache entry, initializing new entries with \ref indeterminate.*/ + tribool & get( sat::Solvable solv_r ) const + { + CacheType::iterator it( _cache.find( solv_r ) ); + if ( it == _cache.end() ) + return (_cache[solv_r] = indeterminate); + return _cache[solv_r]; + } + + mutable CacheType _cache; + mutable AnalyzeStack _stack; + mutable std::string _ltag; +}; + +RunableCache rcache; + +//================================================== + bool upgrade() { bool rres = false; @@ -39,6 +209,7 @@ getZYpp()->resolver()->problems(); return false; } + return true; } @@ -79,70 +250,61 @@ Pathname mroot( "/tmp/Bb" ); TestSetup test( mroot, Arch_ppc64 ); + + ResPool pool( test.pool() ); + sat::Pool satpool( test.satpool() ); + { zypp::base::LogControl::TmpLineWriter shutUp; test.loadTarget(); test.loadTestcaseRepos( "/suse/ma/BUGS/439802/bug439802/YaST2/solverTestcase" ); } + if ( 0 ) + { + PoolItem p = getPi<Package>( "bash", Edition("3.1-24.14"), Arch("ppc64") ); + p.status().setTransact( true, ResStatus::USER ); + } + save(); { zypp::base::LogControl::TmpLineWriter shutUp; getPi<Product>( "SUSE_SLES", Edition("11"), Arch("ppc64") ).status().setTransact( true, ResStatus::USER ); getPi<Package>( "sles-release", Edition("11-54.3"), Arch("ppc64") ).status().setTransact( true, ResStatus::USER ); - //upgrade(); + upgrade(); } - ResPool pool( test.pool() ); vdumpPoolStats( USR << "Transacting:"<< endl, make_filter_begin<resfilter::ByTransact>(pool), make_filter_end<resfilter::ByTransact>(pool) ) << endl; - getZYpp()->resolver()->setIgnoreAlreadyRecommended( true ); - getZYpp()->resolver()->setOnlyRequires( true ); - + pool::GetResolvablesToInsDel collect( pool, pool::GetResolvablesToInsDel::ORDER_BY_MEDIANR ); restore(); - getPi<Package>( "gvfs-backends", Edition("1.0.2-1.4"), Arch("ppc64") ).status().setTransact( true, ResStatus::USER ); - - INT << sat::Solvable(909).requires() << endl; - - vdumpPoolStats( SEC << "Transacting:"<< endl, - make_filter_begin<resfilter::ByTransact>(pool), - make_filter_end<resfilter::ByTransact>(pool) ) << endl; - verify(); - vdumpPoolStats( SEC << "Transacting:"<< endl, + vdumpPoolStats( USR << "Transacting:"<< endl, make_filter_begin<resfilter::ByTransact>(pool), make_filter_end<resfilter::ByTransact>(pool) ) << endl; - INT << "===[END]============================================" << endl << endl; - zypp::base::LogControl::TmpLineWriter shutUp1; - return 0; - - pool::GetResolvablesToInsDel collect( pool, pool::GetResolvablesToInsDel::ORDER_BY_MEDIANR ); { for_( it, collect._toDelete.begin(), collect._toDelete.end() ) { - restore(); it->status().setTransact( true, ResStatus::USER ); SEC << *it << endl; - vdumpPoolStats( SEC << "Transacting:"<< endl, - make_filter_begin<resfilter::ByTransact>(pool), - make_filter_end<resfilter::ByTransact>(pool) ) << endl; - save(); - verify(); +// vdumpPoolStats( SEC << "Transacting:"<< endl, +// make_filter_begin<resfilter::ByTransact>(pool), +// make_filter_end<resfilter::ByTransact>(pool) ) << endl; + + (rcache.isInstallable( *it ) ? MIL : ERR) << *it << " deletable?" << endl; } } - { + if ( 0 ) { for_( it, collect._toInstall.begin(), collect._toInstall.end() ) { - restore(); it->status().setTransact( true, ResStatus::USER ); SEC << *it << endl; - vdumpPoolStats( SEC << "Transacting:"<< endl, - make_filter_begin<resfilter::ByTransact>(pool), - make_filter_end<resfilter::ByTransact>(pool) ) << endl; - save(); - verify(); +// vdumpPoolStats( SEC << "Transacting:"<< endl, +// make_filter_begin<resfilter::ByTransact>(pool), +// make_filter_end<resfilter::ByTransact>(pool) ) << endl; + (rcache.isInstallable( *it ) ? MIL : ERR) << *it << " installable?" << endl; } } -- To unsubscribe, e-mail: zypp-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: zypp-commit+help(a)opensuse.org

1 0