https://bugzilla.novell.com/show_bug.cgi?id=369893 User jkupec@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=369893#c1 Ján Kupec changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jkupec@novell.com Summary|zypper si results in segmentation fault |zypper si does not install build dependencies --- Comment #1 from Ján Kupec 2008-03-16 23:02:49 MST --- I fixed the segfault in r9173 in zypper, but the injecting of the build requires still does not work because srcpkg->dep(Dep::REQUIRES) seems to be empty => hijacking this bug to resolve that -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=369893 User jkupec@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=369893#c3 --- Comment #3 from Ján Kupec 2008-03-17 05:30:11 MST --- Because i thought we don't put srcpackages into the solver pool, do we?. But even if i do zypp->resolver()->addRequire(Capability( srcpkg->name(), Rel::EQ, srcpkg->edition(), ResTraits<SrcPackage>::kind)); i get: Problem: nothing provides requested srcpackage:zypper = 0.10.4-2 Solution 1: do not ask to install a solvable providing srcpackage:zypper = 0.10.4-2 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=369893 User jkupec@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=369893#c5 --- Comment #5 from Ján Kupec 2008-03-17 06:43:18 MST --- Nice, this works via poolitem.status().setToBeInstalled( zypp::ResStatus::USER ) as far as the solver is concerned. $ src/zypper si zypper Reading installed packages... The following NEW packages are going to be installed: libzypp-devel libsatsolver-devel The following NEW srcpackages are going to be installed: zypper But there are three more problems: 1) commit does not install src packages (as we did not deal with src packages through the pool previously) 2) we still need srcpkg->dep(Dep::REQUIRES) to be able to do 'zypper si --no-build-deps foo' (only install the source package, ...well we have ZYpp::installSrcPackage(srcpkg) for that) or to show the build requires to the user. 3) And finally, the build deps are generated only in rpm-md metadata, we don't have them in susetags: ##---------------------------------------- =Pkg: zypper 0.10.3 3 src =Cks: SHA1 e107c279d204938a43182e1c332a029672dc13c5 =Loc: 1 zypper-0.10.3-3.src.rpm src =Siz: 313403 305351 =Shr: zypper 0.10.3 3 ppc So if we are to put the srcpackages into the pool together with the others (i like this approach btw, but i don't know the reasons why we avoided it in the past) we need to fix the above. Note that having the srcpackages in the pool also solves the problem of the query API vs. the srcpackages! -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=369893 User ro@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=369893#c7 --- Comment #7 from Ruediger Oertel 2008-03-17 11:45:31 MST --- no, we do not handle _any_ dependencies for src-rpms currently. (and I'm not really sure it would make sense. 9 out of 10 times you get a src-rpm only to look at it, not to actually compile it). -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=369893 User jkupec@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=369893#c9 --- Comment #9 from Ján Kupec 2008-03-21 08:22:37 MST --- (In reply to comment #6 from Michael Andres)

(In reply to comment #5 from Jan Kupec)

...

2) we still need srcpkg->dep(Dep::REQUIRES) to be able to do <cut> or to show the build requires to the user.

I'd sugest to add a policy flag to the resolver. With no-build-deps we simply won't pass them to the satsolver. (Schubi?)

That would solve the build-deps problem, but we need this also e.g. for zypper info (to show the list of the dependencies of the package), yast needs to display that info too. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=369893 Felix Möller changed: What |Removed |Added ---------------------------------------------------------------------------- Version|Alpha 2plus |Factory -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=369893 User coolo@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=369893#c12 Stephan Kulow changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |rguenther@novell.com --- Comment #12 from Stephan Kulow 2008-05-14 10:35:33 MST --- *** Bug 390344 has been marked as a duplicate of this bug. *** https://bugzilla.novell.com/show_bug.cgi?id=390344 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=369893 User mls@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=369893#c14 --- Comment #14 from Michael Schröder 2008-06-02 02:24:20 MDT --- Btw, there's a way to make the solver select a sourcepackage: you have to create a special dependency that selects only sources, i.e. like "NetworkManager.src". I don't know if libzypp exposes this interface, though. Schubi? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=369893 User jkupec@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=369893#c16 --- Comment #16 from Ján Kupec 2008-06-02 04:22:45 MDT --- comment #14: installing the src rpm only isn't a problem, the old ZYpp::installSrcPackage still works (zypper si -D/--no-build-deps foo). The problem is getting/marking its build deps comment #5, #6. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=369893 User ma@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=369893#c18 --- Comment #18 from Michael Andres 2008-06-03 06:41:41 MDT --- You can find the PoolItem and select it for installation: - Via a Selectable: ui::Selectable::Ptr srcp( getZYpp()->poolProxy().lookup( ResKind::srcpackage, "zypper" ) ); if ( srcp ) srcp->setStatus( S_Install ); - By iterating the pool and manually selecting some available version: PoolItem srcp; for_( it, pool.byIdentBegin( ResKind::srcpackage, "zypper" ), pool.byIdentEnd( ResKind::srcpackage, "zypper" ) ) { if ( it->status().isInstalled() ) continue; if ( !srcp || srcp->edition() < (*it)->edition() ) srcp = *it; } if ( srcp ) srcp.status().setTransact( true, ResStatus::USER ); The only thing that currently does not work is trying to select the source package via: addRequire( Capability(srcpkg->name(), Rel::EQ, srcpkg->edition(), ResTraits<SrcPackage>::kind) ); This is because a source package does not provide its "name == edition". But this will be fixed. Capability will rewrite the capability into something the satsolver understands. --- @Jano: As the current zyyper code (in zypper-misc.cc(2663):build_deps_install) already looks up some appropriate package: SrcPackage::constPtr srcpkg = source_find(*it); the following should select it: if (srcpkg) PoolItem(srcpkg).status().setTransact( true, ResStatus::USER ); No need to drop the item you found, and ask the solver to find it again ;) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=369893 User jkupec@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=369893#c20 Ján Kupec changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |coolo@novell.com Status|ASSIGNED |NEEDINFO Info Provider| |ro@novell.com --- Comment #20 from Ján Kupec 2008-06-05 05:37:55 MDT --- Great, it works now, the source rpm installs fine. But the build deps are still missing in the susetags metadata (see comment #7 and #8). Rpmmd works fine. Rudi, Coolo, can we get this done yet? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=369893 Ján Kupec changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |334207 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=369893 User jkupec@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=369893#c23 Ján Kupec changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution| |FIXED --- Comment #23 from Ján Kupec 2008-06-06 03:18:22 MDT --- Nice, it works now with susetags, too, thanx! I opened new bugs for the remaining issues (bug 397848 and bug 397853) so i'm closing this one. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.