Mailinglist Archive: zypp-commit (545 mails)

[matz@xxxxxxxxxxxxxxxx]

Author: matz
Date: Thu Feb  7 05:10:29 2008
New Revision: 8513

URL: http://svn.opensuse.org/viewcvs/zypp?rev=8513&view=rev
Log:
And now use the new lookupLocation function to get the media path.
With the added hack of prepending "suse/" I can now actually install a
package and it indeed downloads the right rpm.  I've only tried dry
mode, so no actual installation happens.  It seems as if installing just
one package doesn't really trigger a solver run, though (the
dependencies don't seem to be marked as to-install).

Due to the hack with the hard-coded datadir this only works when the
package comes from a susetags repo (the others don't yet have media
locations anyway).

Modified:
    branches/tmp/ma/jump_sat/libzypp/zypp/Package.cc

Modified: branches/tmp/ma/jump_sat/libzypp/zypp/Package.cc
URL: 
http://svn.opensuse.org/viewcvs/zypp/branches/tmp/ma/jump_sat/libzypp/zypp/Package.cc?rev=8513&r1=8512&r2=8513&view=diff
==============================================================================
--- branches/tmp/ma/jump_sat/libzypp/zypp/Package.cc (original)
+++ branches/tmp/ma/jump_sat/libzypp/zypp/Package.cc Thu Feb  7 05:10:29 2008
@@ -106,7 +106,12 @@
   OnMediaLocation Package::location() const
   {
     OnMediaLocation loc;
-    loc.setLocation(lookupStrAttribute( sat::SolvAttr::mediafile ));
+    unsigned medianr;
+    std::string filename = lookupLocation( medianr );
+    /* XXX someone else needs to do this prepending of the datadir.
+       It's not necessarily "suse".  */
+    filename = "suse/" + filename;
+    loc.setLocation(filename, medianr);
     return loc;
   }
 

--
To unsubscribe, e-mail: zypp-commit+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: zypp-commit+help@xxxxxxxxxxxx