]> git.saurik.com Git - apt.git/commitdiff
merged lp:~mvo/apt/mvo
authorMichael Vogt <egon@debian-devbox>
Fri, 12 Oct 2012 12:44:09 +0000 (14:44 +0200)
committerMichael Vogt <egon@debian-devbox>
Fri, 12 Oct 2012 12:44:09 +0000 (14:44 +0200)
1  2 
apt-pkg/contrib/netrc.cc
debian/changelog

Simple merge
index 607dddcfa50b0ac8b17c92e9aab4016cc96fe831,50d6ea3cea49ec2abc4eb69bc3e056aae9882e87..ea7de55d8577a57236f085b8e5b0ac7294fc57e0
 -apt (0.8.16~exp15) unstable; urgency=low
 +apt (0.9.7.6) UNRELEASED; urgency=low
  
 +  [ David Kalnischkies ]
 +  * apt-pkg/pkgcachegen.cc:
 +    - ensure that dependencies for packages:none are always generated
 +    - add 3 missing remap registrations causing a segfault in case
 +      we use the not remapped iterators after a move of the mmap again
 +  * apt-pkg/pkgcache.cc:
 +    - ignore negative dependencies applying in the same group for
 +      M-A:same packages on the real package name as self-conflicts
 +  * cmdline/apt-cache.cc:
 +    - print versioned dependency relations in (r)depends if the option
 +      APT::Cache::ShowVersion is true (default: false) as discussed in
 +      #218995 to help debian-cd fixing #687949. Thanks to Sam Lidder
 +      for initial patch and Steve McIntyre for nagging and testing!
++  
++  [ Michael Vogt ]
++  * increase the maximum netrc login/password size and show proper
++    error message on overflow
 +
 + -- David Kalnischkies <kalnischkies@gmail.com>  Wed, 19 Sep 2012 11:29:56 +0200
 +
 +apt (0.9.7.5) unstable; urgency=low
 +
 +  [ Manpages translation updates ]
 +  * Japanese (KURASAWA Nozomu) (Closes: #684435)
 +  * Portuguese (Américo Monteiro) (Closes: #686975)
 +
 +  [ David Kalnischkies ]
 +  * handle packages without a mandatory architecture (debian-policy §5.3)
 +    by introducing a pseudo-architecture 'none' so that the small group of
 +    users with these packages can get right of them without introducing too
 +    much hassle for other users (Closes: #686346)
 +  * apt-pkg/cdrom.cc:
 +    - copy only configured translation files from a CD-ROM and not all
 +      available translation files preventing new installs with d-i from
 +      being initialized with all translations (Closes: #678227)
 +    - handle Components in the reduction for the source.list as multi-arch CDs
 +      otherwise create duplicated source entries (e.g. "wheezy main main")
 +  * apt-pkg/packagemanager.cc:
 +    - unpack versions only in case a different version from the package
 +      is currently in unpack state to recover from broken system states
 +      (like different file in M-A:same package and other dpkg errors)
 +      and avoid re-unpack otherwise (Closes: #670900)
 +  * debian/control:
 +    - let libapt-pkg break apt < 0.9.4 to ensure that the installed http-
 +      method supports the new redirection-style, thanks to Raphael Geissert
 +      for reporting & testing (Closes: #685192)
 +  * doc/apt_preferences.5.xml:
 +    - use the correct interval (x <= P < y) for pin value documentation as
 +      these are the intervals used by the code (Closes: #685989)
 +  * apt-pkg/indexcopy.cc:
 +    - do not create duplicated flat-archive CD-ROM sources for foreign
 +      architectures on multi-arch CD-ROMs
 +    - do not warn about files which have a record in the Release file, but
 +      are not present on the CD to mirror the behavior of the other methods
 +      and to allow uncompressed indexes to be dropped without scaring users
 +  * apt-pkg/pkgcachegen.cc:
 +    - do not create 'native' (or now 'none') package structures as a side
 +      effect of description translation parsing as it pollutes the cache
 +
 + -- Michael Vogt <mvo@debian.org>  Tue, 11 Sep 2012 15:56:44 +0200
 +
 +apt (0.9.7.4) unstable; urgency=low
 +
 +  [ Manpages translation updates ]
 +  * Polish (Robert Luberda) (Closes: #683109)
 +
 +  [ Program translation updates ]
 +  * Polish (Michał Kułach)
 +
 +  [ Pino Toscano ]
 +  * apt-pkg/contrib/mmap.cc:
 +    - guard only the msync call with _POSIX_SYNCHRONIZED_IO rather
 +      than also the fallback code as it breaks APT on hurd since 0.9.7.3
 +      as the fallback is now always used on non-linux (Closes: #683354)
 +
 +  [ David Kalnischkies ]
 +  * apt-pkg/contrib/fileutl.cc:
 +    - remove _POSIX_SYNCHRONIZED_IO guard in FileFd::Sync() around fsync
 +      as this guard is only needed for fdatasync and not defined on hurd
 +  * cmdline/apt-get.cc:
 +    - error out on (unsatisfiable) build-deps on purly virtual packages
 +      instead of ignoring these dependencies; thanks to Johannes Schauer
 +      for the detailed report! (Closes: #683786)
 +    - ensure that the right architecture is used for cross-dependencies in
 +      cases we have to choose a provider by defaulting on host-arch
 +      instead of build-arch
 +  * doc/apt-verbatim.ent:
 +    - denote 'wheezy' as stable codename and 'jessie' as testing codename
 +      in the documentation in preparation for release
 +  * apt-pkg/indexcopy.cc:
 +    - do not use atomic writing if the target is /dev/null as we don't want
 +      to replace it, not even automically. (Closes: #683410)
 +  * apt-pkg/cdrom.cc:
 +    - do not link() but rename() the cdroms.list to cdroms.list~ as a backup
 +      to ensure that apt-cdrom can be run multiple times (Closes: #676302)
 +
 + -- Michael Vogt <mvo@debian.org>  Mon, 06 Aug 2012 15:55:04 +0200
 +
 +apt (0.9.7.3) unstable; urgency=low
 +
 +  [ Manpages translation updates ]
 +  * Spanish; (Omar Campagne). Closes: #681566
 +
 +  [ Program translation updates ]
 +  * Czech (Miroslav Kure). Closes: #680758
 +
 +  [ David Kalnischkies ]
 +  * apt-pkg/cacheset.cc:
 +    - handle :all and :native correctly as architectures again
 +      in the commandline parsing (regression in 0.9.7)
 +  * apt-pkg/packagemanager.cc:
 +    - do not segfault if nothing can be configured to statisfy
 +      a pre-depends (e.g. in a pre-depends loop) (Closes: #681958)
 +  * apt-pkg/contrib/mmap.cc:
 +    - trigger the usage of the fallback code for kfreebsd also in the
 +      second (filebased) constructor of DynamicMMap (Closes: #677704)
 +    - refer to APT::Cache-Start in case the growing failed as if -Limit is
 +      really the offender it will be noted in a previous error message.
 +    - for filesystems not supporting mmap'ing a file we need to use a
 +      SyncToFd dummy just as we did for compressed files in 0.9.5
 +
 + -- Michael Vogt <mvo@debian.org>  Fri, 27 Jul 2012 17:53:41 +0200
 +
 +apt (0.9.7.2) unstable; urgency=low
 +
 +  [ Manpages translation updates ]
 +  * French (Christian Perrier)
 +  * German (Chris Leick)
 +
 +  [ Program translation updates ]
 +  * Greek (Θανάσης Νάτσης)
 +  * Japanese (Kenshi Muto) (Closes: #679662)
 +  * Russian (Yuri Kozlov) (Closes: #679599)
 +  * Danish (Joe Dalton) (Closes: #680119)
 +  * Portuguese (Miguel Figueiredo) (Closes: #680616)
 +
 +  [ David Kalnischkies ]
 +  * debian/apt.cron.daily:
 +    - do not try to backup extended_states file if it doesn't
 +      exist (Closes: #680287)
 +  * ftparchive/writer.cc:
 +    - handle the APT::FTPArchive::Packages::SHA512 option correctly instead
 +      of overriding SHA256, thanks Christian Marillat! (Closes: #680252)
 +  * cmdline/apt-mark.cc:
 +    - arch:all packages are treated as arch:native packages, but dpkg
 +      expects pkg:all for selections, so use the arch of the installed
 +      version instead of the package structure if possible.
 +      Thanks to Stepan Golosunov for the report! (Closes: #680041)
 +  * apt-pkg/clean.cc:
 +    - run autoclean against pkg:arch and not always against pkg:native as
 +      this removes valid cache entries (Closes: #679371)
 +  * apt-pkg/deb/deblistparser.cc:
 +    - negative dependencies need to apply to all architectures,
 +      but those with a specific architecture only apply to this one
 +  * apt-pkg/cachefilter.cc:
 +    - remove architecture-specific arch to tuple expansion-rules as they lead
 +      to the same tuples for different architectures (e.g. linux-arm for arm,
 +      armel and armhf) while the dpkg-architecture code uses triples which
 +      are different (in the first part, which we omit in our tuples), so e.g.
 +      build-dep restrictions for armel ended up effecting armhf as well
 +
 + -- Michael Vogt <mvo@debian.org>  Fri, 13 Jul 2012 21:33:56 +0200
 +
 +apt (0.9.7.1) unstable; urgency=low
 +
 +  [ Program translation updates ]
 +  * Bulgarian (Damyan Ivanov) (Closes: #678983)
 +  * Hungarian (Gabor Kelemen)
 +  * Italian (Milo Casagrande)
 +  * Slovenian (Andrej Znidarsic)
 +  * German (Holger Wansing) (Closes: #679314)
 +  * Slovak (Ivan Masár) (Closes: #679448)
 +
 +  [ David Kalnischkies ]
 +  * cmdline/apt-internal-solver.cc, cmdline/apt-mark.cc:
 +    - typo fixes and unfuzzy translations
 +  * debian/control:
 +    - libapt-{pkg,inst} packages should be in section 'libs' instead
 +      of 'admin' as by ftp-master override request in #677596
 +    - demote debiandoc-sgml to Build-Depends-Indep
 +  * doc/makefile:
 +    - separate translation building of debiandoc from manpages
 +      so that we don't need to build debiandoc for binary packages
 +
 + -- Michael Vogt <mvo@debian.org>  Fri, 29 Jun 2012 14:26:32 +0200
 +
 +apt (0.9.7) unstable; urgency=low
 +
 +  [ Julian Andres Klode ]
 +  * apt-pkg/contrib/mmap.cc:
 +    - Fix the Fallback option to work correctly, by not calling
 +      realloc() on a map mapped by mmap(), and by using malloc
 +      and friends instead of new[].
 +    - Zero out the new memory allocated with realloc().
 +
 +  [ Daniel Hartwig ]
 +  * apt-pkg/pkgcachegen.cc:
 +    - always reset _error->StackCount in MakeStatusCache (Closes: #677175)
 +
 +  [ David Kalnischkies ]
 +  * apt-pkg/deb/deblistparser.cc:
 +    - ensure that mixed positive/negative architecture wildcards
 +      are handled in the same way as dpkg handles them
 +    - use PackageArchitectureMatchesSpecification filter
 +  * apt-pkg/cachefilter.cc:
 +    - add PackageArchitectureMatchesSpecification (Closes: #672603)
 +  * apt-pkg/cacheset.cc:
 +    - add PackageContainerInterface::FromGroup to support
 +      architecture specifications with wildcards on the commandline
 +  * apt-pkg/pkgcache.cc:
 +    - do a string comparision for architecture checking in IsMultiArchImplicit
 +      as 'unique' strings in the pkgcache aren't unique (Closes: #677454)
 +  * buildlib/configure.mak:
 +    - print a message detailing how to get config.guess and config.sub
 +      in case they are not in /usr/share/misc (Closes: #677312)
 +  * cmdline/apt-get.cc:
 +    - print a friendly message in 'download' if a package can't be
 +      downloaded (Closes: #677887)
 +
 + -- Michael Vogt <mvo@debian.org>  Tue, 19 Jun 2012 16:42:43 +0200
 +
 +apt (0.9.6) unstable; urgency=low
 +
 +  [ David Kalnischkies ]
 +  * apt-pkg/cdrom.cc:
 +    - fix regression from 0.9.3 which dumped the main configuration
 +      _config instead of the cdrom settings (Cnf) as identified and
 +      tested by Milan Kupcevic, thanks! (Closes: #674100)
 +  * cmdline/apt-get.cc:
 +    - do not show 'list of broken packages' header if no package
 +      is broken as it happens e.g. for external resolver errors
 +    - print URIs for all changelogs in case of --print-uris,
 +      thanks to Daniel Hartwig for the patch! (Closes: #674897)
 +    - show 'bzr branch' as 'bzr get' is deprecated (LP: #1011032)
 +    - check build-dep candidate if install is forbidden
 +  * debian/apt-utils.links:
 +    - the internal resolver 'apt' is now directly installed in
 +      /usr/lib/apt/solvers, so don't instruct dh to create a broken link
 +  * doc/apt-verbatim.ent:
 +    - APT doesn't belong to the product 'Linux', so use 'APT' instead
 +      as after all APT is a big suite of applications
 +  * doc/examples/sources.list:
 +    - use the codename instead of 'stable' in the examples sources.list
 +      as we do in the manpage and as the debian-installer does
 +  * doc/apt-get.8.xml:
 +    - use apt-utils as package example instead of libc6
 +  * apt-pkg/contrib/cmdline.cc:
 +    - apply patch from Daniel Hartwig to fix a segfault in case
 +      the LongOpt is empty (Closes: #676331)
 +    - fix segfault with empty LongOpt in --no-* branch
 +  * ftparchive/apt-ftparchive.cc:
 +    - default to putting the Contents-* files below $(SECTION) as apt-file
 +      expects them there - thanks Martin-Éric Racine! (Closes: #675827)
 +  * apt-pkg/deb/deblistparser.cc:
 +    - set pkgCacheGen::Essential to "all" again (Closes: #675449)
 +  * apt-pkg/algorithms.cc:
 +    - force install only for one essential package out of a group
 +  * apt-pkg/aptconfiguration.cc:
 +    - if APT::Languages=none save "none" in allCodes so that the detected
 +      configuration is cached as intended (Closes: #674690, LP: #1004947)
 +  * apt-pkg/cacheiterators.h:
 +    - add an IsMultiArchImplicit() method for Dep- and PrvIterator
 +
 +  [ Justin B Rye ]
 +  * doc/apt-cdrom.8.xml:
 +    - replace CDROM with the proper CD-ROM in text
 +    - correct disc vs. disk issues
 +  * doc/apt-extracttemplates.1.xml:
 +    - debconf is not DebConf
 +  * doc/apt-get.8.xml:
 +    - move dselect-upgrade below dist-upgrade
 +    - review and fix spelling issues
 +  * doc/apt-ftparchive.8.xml, doc/apt-config.8.xml,
 +    doc/apt-key.8.xml, doc/apt-mark.8.xml,
 +    doc/apt_preferences.5.xml, doc/apt-secure.8.xml,
 +    doc/apt-sortpkgs.1.xml, sources.list.5.xml:
 +    - review and fix typo, grammar and style issues
 +  * doc/apt.conf.5.xml:
 +    - review and fix typo, grammar and style issues
 +    - rephrase APT::Immediate-Configuration and many others
 +
 +  [ Sebastian Heinlein ]
 +  * cmdline/apt-key:
 +    - do not hardcode /etc but use Dir::Etc instead
 +
 +  [ Robert Luberda ]
 +  * Polish manpage translation update (Closes: #675603)
 +  * doc/apt-mark.8.xml:
 +    - in hold, the option name is --file not --filename
 +
 +  [ Christian Perrier ]
 +  * French program and manpage translation update
 +  * Danish program translation by Joe Hansen. Closes: #675605
 +
 +  [ Thibaut Girka ]
 +  * cmdline/apt-get.cc:
 +    - complain correctly about :any build-dep on M-A:none packages
 +  * apt-pkg/deb/deblistparser.cc:
 +    - add support for arch-specific qualifiers in dependencies
 +
 + -- Michael Vogt <mvo@debian.org>  Mon, 11 Jun 2012 16:21:53 +0200
 +
 +apt (0.9.5.1) unstable; urgency=low
 +
 +  [ David Kalnischkies ]
 +  * apt-pkg/contrib/fileutl.cc:
 +    - dup() given compressed fd in OpenDescriptor if AutoClose
 +      is disabled as otherwise gzclose() and co will close it
 +  * doc/*.xml:
 +    - mark even more stuff as untranslateable and improve the
 +      markup here and there (no real text change)
 +    - use docbook DTD 4.5 instead of 4.2 to have valid docs
 +
 +  [ Justin B Rye ]
 +  * doc/*.xml:
 +    - remove 'GNU/Linux' from 'Debian systems' strings as Debian
 +      has more systems than just GNU/Linux nowadays
 +  * doc/apt-cache.8.xml:
 +    - fix a typo as well as adding missing literal markup
 +    - three small rewordings for better english sentences
 +
 + -- Michael Vogt <mvo@debian.org>  Thu, 24 May 2012 17:16:34 +0200
 +
 +apt (0.9.5) unstable; urgency=low
 +
 +  [ Chris Leick ]
 +  * proofreading of the manpage pot
 +  * German manpage translation update (Closes: #673294)
 +
 +  [ David Kalnischkies ]
 +  * buildlib/podomain.mak:
 +    - ensure that all sources end up in the srclist so that we don't
 +      forget to extract half of the translation strings
 +  * buildlib/inttypes.h.in:
 +    - remove inttypes.h compatibility as providing such a c99 types
 +      compatibility conflicts with the usage of c99 type long long
 +  * apt-pkg/contrib/mmap.cc:
 +    - have a dummy SyncToFd around in case of ReadOnly access to a
 +      compressed file as we otherwise on Close() do not delete[] the
 +      char buffer but munmap() it… (Closes: #673815)
 +  * debian/control:
 +    - moving debiandoc-sgml to Build-Depends-Indep was one step too much
 +      for the buildds as we still build two sgml files in arch:any
 +  * debian/rules:
 +    - move internal-solver as 'apt' to his friend dump-solver in
 +      /usr/lib/apt/solvers to avoid writing a manpage for it
 +
 + -- Michael Vogt <mvo@debian.org>  Tue, 22 May 2012 16:14:22 +0200
 +
 +apt (0.9.4) unstable; urgency=low
 +
 +  [ David Kalnischkies ]
 +  * methods/http.cc:
 +    - after many years of pointless discussions disable http/1.1 pipelining
 +      by default as many webservers and proxies seem to be unable to conform
 +      to specification must's (rfc2616 section 8.1.2.2) (LP: #996151)
 +    - add spaces around PACKAGE_VERSION to fix FTBFS with -std=c++11
 +  * apt-pkg/pkgcachegen.cc:
 +    - make IsDuplicatedDescription static so that it is really private
 +      as we don't need a symbol for it as it is not in a header
 +  * Makefile, buildlib/*.mak:
 +    - reshuffle dependencies so that parallel building seems to work
 +    - separate manpages from the rest of the doc building
 +  * prepare-release:
 +    - apt-inst version isn't apt versions, so don't override variable
 +  * debian/rules:
 +    - apt-utils packages manpages, so it should depend on build-doc
 +    - make apt and apt-utils packages depend on manpages instead of full doc
 +  * debian/control:
 +    - move doxygen and debiandoc-sgml to Build-Depends-Indep as docs
 +      are no longer build in the same target as the manpages
 +  * apt-pkg/acquire-methods.cc:
 +    - factor out into private Dequeue() to fix access to deleted pointer
 +  * apt-pkg/contrib/fileutl.cc:
 +    - ensure that we close compressed fds, wait for forks and such even if
 +      the FileFd itself is set to not autoclose the given Fd
 +  * cmdline/apt-get.cc:
 +    - use the host architecture, not the build architecture for matching
 +      of [architecture restrictions] in Build-Depends (Closes: #672927)
 +  * doc/makefile:
 +    - build manpages with the correct l10n.gentext.default.language setting
 +      to get the correct section titles provided by docbook
 +  * doc/po/de.po:
 +    - updated german manpage translation by Chris Leick, thanks!
 +  * apt-pkg/packagemanager.cc:
 +    - do not run into loop on new-pre-depends-breaks (Closes: #673536)
 +  * doc/*.xml:
 +    - add a few translator notes and reword some paragraphs to ensure that
 +      translators and users alike can better understand them (Closes: #669409)
 +    - in <term> mark all options with <option> and mark <term><option>
 +      as untranslated for po4a removing ~200 unless "translateable" strings
 +  * apt-pkg/aptconfiguration.cc:
 +    - longcode Translation files are saved with encoded underscore,
 +      so make sure to pick these files up as well for Acquire::Languages
 +  * ftparchive/writer.cc:
 +    - include Contents-* files in Release files (Closes: #673647)
 +
 +  [ Michael Vogt ]
 +  * merged updated de.po, thanks to Holger Wansing (closes: #672466)
 +  
 +  [ Raphael Geissert ]
 +  * apt-pkg/acquire*.cc:
 +    - handle redirections in the worker with the right method instead of
 +      in the method the redirection occured in (Closes: #668111)
 +  * methods/http.cc:
 +    - forbid redirects to change protocol
 +  * methods/mirror.cc:
 +    - generate an equal sign also for the first arch (Closes: #669142)
 +
 +  [ Marius Vollmer ]
 +  * apt-pkg/algorithms.cc:
 +    - fix memory leak of Flags in pkgSimulate by a proper destructor
 +
 + -- Michael Vogt <mvo@debian.org>  Mon, 21 May 2012 12:29:05 +0200
 +
 +apt (0.9.3) unstable; urgency=low
 +
 +  [ David Kalnischkies ]
 +  * apt-pkg/contrib/strutl.cc:
 +    - remove the message size limit from ioprintf and strprintf
 +  * apt-pkg/contrib/configuration.cc:
 +    - add a more versatile Dump() method
 +    - normalize a bit by replacing // and /./ with / in FindFile
 +    - /dev/null is a special absolute path as it has no subdirectories
 +  * apt-pkg/acquire-worker.cc:
 +    - use Dump() to generate the configuration message for sending
 +  * cmdline/apt-config.cc:
 +    - make it possible to limit dump to a subtree
 +    - implement --empty and --format option for dump
 +  * apt-pkg/cdrom.cc:
 +    - use Dump() to generate the configuration output
 +  * apt-pkg/depcache.cc:
 +    - clearly separate 'positive' and 'negative' dependencies and
 +      their upgrade-resolution tries in MarkInstall and especially don't
 +      treat Conflicts differently compared to Breaks here
 +    - provider is only a possible solution if the provides has the right
 +      version (or none as we have no versioned provides in debian) and not
 +      if the version of the provider matches
 +  * edsp/edspsystem.cc:
 +    - check with RealFileExists for scenario file as otherwise a directory
 +      like one provided with RootDir triggers the usage of EDSP
 +  * debian/libapt-inst1.5.symbols:
 +    - use the correct library name the symbols header
 +  * apt-pkg/pkgcachegen.cc:
 +    - check if NewDescription allocation has failed and error out accordingly
 +    - check if we work on a valid description in IsDuplicateDescription as
 +      we end up working on dangling pointers otherwise which segfaults on
 +      s390x and ppc64 (Closes: #669427)
 +  * apt-pkg/deb/deblistparser.cc:
 +    - check length and containing chars for a given description md5sum
 +  * ensure that apti18n.h is included last as advertised (Closes: #671623)
 +  * apt-pkg/acquire-worker.cc:
 +    - revert the use of FileFd::Write in OutFdReady as we don't want error
 +      reports about EAGAIN here as we retry later. Thanks to YOSHINO Yoshihito
 +      for the report. (Closes: #671721)
 +  * apt-pkg/contrib/fileutl.cc:
 +    - check that the fd which are closed are valid
 +    - ensure that we do init d only once and especially not with its own
 +      content as this causes some "interesting" hickups resulting in segfaults
 +      as it seems (Closes: #554387, #670979)
 +    - collect zombie (de)compressor processes on reopen
 +    - ensure that in error conditions the Fail flag is set
 +    - ensure that d is set before accessing it
 +  * apt-pkg/aptconfiguration.cc:
 +    - use NULL instead of "" for no (un)compress parameters
 +  * apt-pkg/algorithms.cc:
 +    - factor out of ListUpdate a AcquireUpdate to be able to provide your
 +      own pkgAcquire fetcher to the wrapper
 +  * apt-inst/deb/debfile.h:
 +    - readd 'md5.h' to the uncleaned header includes to make qapt build
 +      against us again unchanged to unblock transition (Closes: #669163)
 +
 + -- Michael Vogt <mvo@debian.org>  Fri, 11 May 2012 17:16:22 +0200
 +
 +apt (0.9.2) unstable; urgency=low
 +
 +  [ Michael Vogt ]
    * apt-inst/contrib/extracttar.cc:
      - ensure that in StartGzip the InFd is set to "AutoClose" to ensure
        that the pipe is closed when InFd is closed. This fixes a Fd leak