Browse Source

move manual-bit from 'oldlibs' pkg to its dependencies

oldlibs used to be in APT::Never-MarkAuto-Sections so that old
transition packages can be removed without causing the then
(autoinstalled) renamed package to be autoremoved. It isn't ideal
through as ideally you want the oldlibs package to be removed after
nothing depends on it anymore regardless of if you have once installed
it by hand or not – and if you had the package talking over (the
dependencies) should carry the manual bit now as they are the real deal
now.

As an added bonus if the package has no dependencies because it is an
oldlibs without a direct replacement you should move away from (like
lib1 and lib2 are currently in the archive, but there will hopefully
only be lib2 in the release) you get a lib1 marked as auto.

If the user still needs the oldlibs package for some reason all he has
to do is mark it as manual once as this move is only performed if a
installed package changes its section from a not-Move-Autobit-Sections
to a Move-Autobit-Sections.
tags/debian/1.1.exp9
David Kalnischkies 6 years ago
parent
commit
5f4495e342
3 changed files with 63 additions and 0 deletions
  1. +28
    -0
      apt-pkg/depcache.cc
  2. +4
    -0
      debian/apt.conf.autoremove
  3. +31
    -0
      test/integration/test-apt-move-and-forget-manual-sections

+ 28
- 0
apt-pkg/depcache.cc View File

@@ -1128,9 +1128,26 @@ bool pkgDepCache::MarkInstall(PkgIterator const &Pkg,bool AutoInst,
if (DebugMarker == true)
std::clog << OutputInDepth(Depth) << "MarkInstall " << Pkg << " FU=" << FromUser << std::endl;

bool MoveAutoBitToDependencies = false;
VerIterator const PV = P.InstVerIter(*this);
if (unlikely(PV.end() == true))
return false;
else if (PV->Section != 0 && (P.Flags & Flag::Auto) != Flag::Auto)
{
VerIterator const CurVer = Pkg.CurrentVer();
if (CurVer.end() == false && CurVer->Section != 0 && strcmp(CurVer.Section(), PV.Section()) != 0)
{
bool const CurVerInMoveSection = ConfigValueInSubTree("APT::Move-Autobit-Sections", CurVer.Section());
bool const InstVerInMoveSection = ConfigValueInSubTree("APT::Move-Autobit-Sections", PV.Section());
MoveAutoBitToDependencies = (CurVerInMoveSection == false && InstVerInMoveSection == true);
if (MoveAutoBitToDependencies == true)
{
if(DebugAutoInstall == true)
std::clog << OutputInDepth(Depth) << "Setting " << Pkg.FullName(false) << " as auto-installed, moving manual to its dependencies" << std::endl;
MarkAuto(Pkg, true);
}
}
}

DepIterator Dep = PV.DependsList();
for (; Dep.end() != true;)
@@ -1244,6 +1261,17 @@ bool pkgDepCache::MarkInstall(PkgIterator const &Pkg,bool AutoInst,
verlist.erase(InstVer);
continue;
}

// now check if we should consider it a automatic dependency or not
if(InstPkg->CurrentVer == 0 && MoveAutoBitToDependencies)
{
if(DebugAutoInstall == true)
std::clog << OutputInDepth(Depth) << "Setting " << InstPkg.FullName(false) << " NOT as auto-installed (direct "
<< Start.DepType() << " of " << Pkg.FullName(false) << " which is manual and in APT::Move-Autobit-Sections)" << std::endl;
MarkAuto(InstPkg, false);
}


break;
} while(true);
continue;


+ 4
- 0
debian/apt.conf.autoremove View File

@@ -32,6 +32,10 @@ APT
"restricted/metapackages";
"universe/metapackages";
"multiverse/metapackages";
};

Move-Autobit-Sections
{
"oldlibs";
"restricted/oldlibs";
"universe/oldlibs";


+ 31
- 0
test/integration/test-apt-move-and-forget-manual-sections View File

@@ -0,0 +1,31 @@
#!/bin/sh
set -e

TESTDIR=$(readlink -f $(dirname $0))
. $TESTDIR/framework
setupenvironment
configarchitecture 'amd64'

aptconfig dump --no-empty --format '%v%n' APT::Move-Autobit-Sections > move-autobit.sections
testsuccess grep '^oldlibs$' move-autobit.sections

buildsimplenativepackage 'libabc' 'amd64' '1' 'stable' '' '' 'libs'
buildsimplenativepackage 'libabc' 'amd64' '2' 'unstable' 'Depends: libdef' '' 'oldlibs'
buildsimplenativepackage 'libdef' 'amd64' '1' 'unstable' '' '' 'libs'
setupaptarchive

testmarkedauto
testmarkedmanual

testsuccess aptget install libabc/stable -y
testdpkginstalled 'libabc'
testdpkgnotinstalled 'libdef'

testmarkedmanual 'libabc'
testmarkedauto

testsuccess aptget dist-upgrade -y
testdpkginstalled 'libabc' 'libdef'

testmarkedauto 'libabc'
testmarkedmanual 'libdef'

Loading…
Cancel
Save