Bug #329060 “Please disable “debian-changelog-file-is-a-symlink”...” : Bugs : lintian package : Ubuntu

Revision history for this message

Adrien Cunin (adri2000) wrote on 2009-02-23:

#1

"Since the changelog symlinks is enabled by Ubuntu" - what do you mean exactly?

Is changelog being a symlink forbidden in Debian? Is it different in Ubuntu?
(I didn't find anything about this in the Debian policy)

Changed in lintian:
status:	New → Incomplete

Revision history for this message

to be removed (liw) wrote on 2009-02-23:

#2

Yes, Debian forbids the changelog from being a symlink, while Ubuntu doesn't. I removed the "invalid" status because of this. (What's the point of asking a question and immediately marking the bug as invalid, before anyone has had a chance to answer?)

Changed in lintian:
status:	Incomplete → New
status:	New → Confirmed

Revision history for this message

Adrien Cunin (adri2000) wrote on 2009-02-23:

#3

Invalid? I marked the bug as incomplete, not invalid. And it's still incomplete.

My questions were implicitly asking for references. In which documents is it written that "Debian forbids the changelog from being a symlink, while Ubuntu doesn't" ?

Changed in lintian:
status:	Confirmed → Incomplete

Revision history for this message

to be removed (liw) wrote on 2009-02-23:

#4

Oh, my error: I misread, you did indeed use incomplete. My apologies.

To expand on my answer: it is not clearly documented in Debian, but lintian is correct in that symlinking the changelog file to another package is treated as a bug in Debian: a changelog is a required part of each package, and symlinking doesn't mean the file is in the package. It's possible to get into a situation where the symlink target isn't installed, and then the changelog isn't available at all for that package.

Ubuntu does not have a problem with this, and Ubuntu's build daemons replace identical copies of changelogs with symlinks to one.

Changed in lintian:
status:	Incomplete → Confirmed

Revision history for this message

Morten Kjeldgaard (mok0) wrote on 2009-02-26:

#5

Right. So people building packages on Ubuntu can't avoid this warning, and I've seen some that attempt to override it, which evidently is a packaging bug. That's why the tag should be disabled in Ubuntu's version of Lintian.

Revision history for this message

Yan Li (yanli) wrote on 2009-09-15:

#6

This bug is supposed to be fixed in Debian's lintian 2.2.15. Please sync as I still see this issue in Karmic, which is using lintian 2.2.14.

Revision history for this message

Russ Allbery (rra-debian) wrote on 2009-09-15: Re: [Bug 329060] Re: Please disable "debian-changelog-file-is-a-symlink" warning

#7

Yan Li <email address hidden> writes:

> This bug is supposed to be fixed in Debian's lintian 2.2.15. Please sync
> as I still see this issue in Karmic, which is using lintian 2.2.14.

Note that it's only fixed if the version number of the package contains
the string "ubuntu." For Ubuntu native packages, there unfortunately
isn't any way for Lintian to know that it's an Ubuntu package.

--
Russ Allbery (<email address hidden>) <http://www.eyrie.org/~eagle/>

Revision history for this message

Benjamin Drung (bdrung) wrote on 2009-09-16:

#8

"lsb_release -si" should be used. I will reopen the Debian bug for this.

Revision history for this message

Benjamin Drung (bdrung) wrote on 2009-09-16:

#9

I have added a fix for this bug to the merge request (bug #430658).

Revision history for this message

Launchpad Janitor (janitor) wrote on 2009-09-24:

#10

Download full text (9.7 KiB)

This bug was fixed in the package lintian - 2.2.17ubuntu1

---------------
lintian (2.2.17ubuntu1) karmic; urgency=low

* Don't warn about a symlinked changelog file in Ubuntu (LP: #329060).

lintian (2.2.17) unstable; urgency=medium

* Urgency medium to beat libclass-accessor-perl into testing.

  * checks/control-file:
    + [RA] Correctly exclude lib*-dev dependencies on other lib*-dev
      packages that contain a version number. Thanks, Yves-Alexis Perez.
      (Closes: #547773)

  * debian/control:
    + [RA] Explicitly depend on libclass-accessor-perl, since we use it
      directly. (libparse-debianchangelog-perl was implicitly ensuring it
      was installed.)

lintian (2.2.16) unstable; urgency=low

  * Summary of tag changes:
    + Added:
      - dh_undocumented-is-obsolete
      - lib-recommends-documentation
      - weak-library-dev-dependency

  * checks/control-file{,.desc}:
    + [RA] Check that -dev packages depend on exactly the same version of
      the shared library package. (Closes: #537768)
  * checks/debhelper{,.desc}:
    + [RA] Check for use of the obsolete dh_undocumented debhelper
      program. Patch from Raphael Geissert. (Closes: #541977)
  * checks/fields{,.desc}:
    + [RA] Check for library packages that recommend documentation
      packages. Patch from Raphael Geissert. (Closes: #527363)
    + [RA] Fix regex for recognizing CVS repositories available via
      anonymous ssh. Thanks, Thorsten Glaser. (Closes: #546927)
  * checks/menus{,.desc}:
    + [RA] Exclude examples from documentation that may require doc-base
      registration. Thanks, Mikhail Gusarov. (Closes: #547345)
  * checks/scripts{,.desc}:
    + [RA] Add /usr/bin/ir as an interpreter provided by the ironruby
      package. Thanks, Matthias Klose. (Closes: #547555)
    + [ADB] Fix a typo in the description of example-script-uses-bin-env
      (pth -> path). Thanks, Jakub Wilk. (Closes: #547670)

  * lib/Lintian/Output.pm:
    + [ADB] Import Exporter before Class::Accessor to avoid accidentally
      calling Class::Accessor::import(). Thanks, Niko Tyni.
      (Closes: #547631)
  * lib/Spelling.pm:
    + [RA] Additional spelling corrections from Raphael Geissert.
    + [RA] Return the count of spelling errors and support suppressing the
      tag by passing undef for the tag name so that the spell checking
      functions can be used to test tag descriptions for spelling errors.
      (Closes: #546641)

  * reporting/html_reports:
    + [RA] Include all tag types in the qa-list.txt file instead of just
      errors and warnings. The new order of columns is errors, warnings,
      info tags, pedantic tags, experimental tags, and overridden tags.

lintian (2.2.15) unstable; urgency=low

The "BTS cleanup" release.

  * Summary of tag changes:
    + Added:
      - build-depends-on-non-build-package
      - desktop-entry-limited-to-environments
      - example-interpreter-in-usr-local (pedantic)
      - example-interpreter-not-absolute
      - example-script-without-interpreter
      - example-script-uses-bin-env
      - example-shell-script-fails-syntax-check (pedantic)
      - example-wrong-path-for-interpreter
      - example...

This bug was fixed in the package lintian - 2.2.17ubuntu1

---------------
lintian (2.2.17ubuntu1) karmic; urgency=low

* Don't warn about a symlinked changelog file in Ubuntu (LP: #329060).

lintian (2.2.17) unstable; urgency=medium

* Urgency medium to beat libclass-accessor-perl into testing.

* checks/control-file:
    + [RA] Correctly exclude lib*-dev dependencies on other lib*-dev
      packages that contain a version number.  Thanks, Yves-Alexis Perez.
      (Closes: #547773)

* debian/control:
    + [RA] Explicitly depend on libclass-accessor-perl, since we use it
      directly.  (libparse-debianchangelog-perl was implicitly ensuring it
      was installed.)

lintian (2.2.16) unstable; urgency=low

* Summary of tag changes:
    + Added:
      - dh_undocumented-is-obsolete
      - lib-recommends-documentation
      - weak-library-dev-dependency

* checks/control-file{,.desc}:
    + [RA] Check that -dev packages depend on exactly the same version of
      the shared library package.  (Closes: #537768)
  * checks/debhelper{,.desc}:
    + [RA] Check for use of the obsolete dh_undocumented debhelper
      program.  Patch from Raphael Geissert.  (Closes: #541977)
  * checks/fields{,.desc}:
    + [RA] Check for library packages that recommend documentation
      packages.  Patch from Raphael Geissert.  (Closes: #527363)
    + [RA] Fix regex for recognizing CVS repositories available via
      anonymous ssh.  Thanks, Thorsten Glaser.  (Closes: #546927)
  * checks/menus{,.desc}:
    + [RA] Exclude examples from documentation that may require doc-base
      registration.  Thanks, Mikhail Gusarov.  (Closes: #547345)
  * checks/scripts{,.desc}:
    + [RA] Add /usr/bin/ir as an interpreter provided by the ironruby
      package.  Thanks, Matthias Klose.  (Closes: #547555)
    + [ADB] Fix a typo in the description of example-script-uses-bin-env
      (pth -> path).  Thanks, Jakub Wilk.  (Closes: #547670)

* lib/Lintian/Output.pm:
    + [ADB] Import Exporter before Class::Accessor to avoid accidentally
      calling Class::Accessor::import().  Thanks, Niko Tyni.
      (Closes: #547631)
  * lib/Spelling.pm:
    + [RA] Additional spelling corrections from Raphael Geissert.
    + [RA] Return the count of spelling errors and support suppressing the
      tag by passing undef for the tag name so that the spell checking
      functions can be used to test tag descriptions for spelling errors.
      (Closes: #546641)

* reporting/html_reports:
    + [RA] Include all tag types in the qa-list.txt file instead of just
      errors and warnings.  The new order of columns is errors, warnings,
      info tags, pedantic tags, experimental tags, and overridden tags.

lintian (2.2.15) unstable; urgency=low

The "BTS cleanup" release.

* Summary of tag changes:
    + Added:
      - build-depends-on-non-build-package
      - desktop-entry-limited-to-environments
      - example-interpreter-in-usr-local (pedantic)
      - example-interpreter-not-absolute
      - example-script-without-interpreter
      - example-script-uses-bin-env
      - example-shell-script-fails-syntax-check (pedantic)
      - example-wrong-path-for-interpreter
      - example-unusual-interpreter (pedantic)
      - global-data-in-games-directory
      - init.d-script-does-not-provide-itself
      - init.d-script-possible-missing-stop
      - init.d-script-provides-virtual-facility
      - init.d-script-should-depend-on-virtual-facility
      - init.d-script-starts-in-stop-runlevel
      - ocaml-custom-executable

* checks/binaries{,.desc}:
    + [RA] Add a new check for OCaml bytecode executables.  Patch from
      Stéphane Glondu.  (Closes: #498138)
    + [RA] Allow ldconfig to be statically linked.
  * checks/changelog-file:
    + [RA] Don't warn about a symlinked changelog file if the version
      contains ubuntu.  Thanks, Benjamin Drung.  (Closes: #536434)
  * checks/common_data.pm:
    + [RA] Moved the essential package list to data/fields.
  * checks/copyright-file:
    + [RA] Use Lintian::Data to read the essential package list.
  * checks/debhelper:
    + [RA] Ignore whitespace when checking debian/compat, since Perl will
      when converting it to a number.  Thanks, Niels Thykier.
      (Closes: #545430)
  * checks/fields{,.desc}:
    + [RA] Exclude base-files from the needlesly-depends-on-awk tag
      and from the warning about depending on a virtual package without
      a real package for the awk virtual package.
    + [RA] Use Lintian::Data to read the essential package list.
    + [RA] Stop allowing unversioned build dependencies on coreutils.  The
      migration from fileutils is long since finished.
    + [RA] Check for build dependencies on packages that should not be
      used for builds.  (Closes: #540594)
    + [RA] Allow architecture-dependent metapackages, since it can be
      useful for metapackages to change dependencies based on the
      architecture.  Thanks, Vagrant Cascadian.  (Closes: #544004)
  * checks/files{,.desc}:
    + [RA] Suppress non-standard-dir-perm for /tmp, /var/tmp, /var/local,
      /var/lock, and /usr/src if they have their normal owner and
      permissions.  This suppresses some Lintian tags for base-files.
    + [RA] Allow base-files to ship /var/backups and /var/local.
    + [RA] Remove swfobject.js from the known packaged Javascript
      libraries.  The version in libjs-yui is old and the new version is
      not packaged.  Thanks, Giuseppe Iuculano.  (Closes: #542046)
    + [RA] Allow t1- and xfonts- as prefixes for font packages since the
      font regex is also looking for Type 1 fonts.  Thanks, Rogério
      Brito.  (Closes: #542389)
    + [RA] Stop looking for *.pcf files outside font packages.  The check
      wasn't triggering since PCF files are always compressed, and the
      situation for PCF font packages is more complex and not suitable for
      the same check.
    + [RA] Add mootools.js to embedded-javascript-library.  Patch from
      Marcelo Jorge Vieira (metal).  (Closes: #545716)
    + [RA] Check for putting data that should be in /usr/share under
      /usr/share/games.  Patch from Raphael Geissert.  (Closes: #534486)
  * checks/init.d{,.desc}:
    + [RA] Add new checks for starting in runlevels 0 and 6, providing a
      virtual facility, not providing a facility matching the script, not
      depending on a virtual facility, and inconsistent stop runlevels.
      Patch from Raphael Geissert.  (Closes: #544221)
  * checks/manpages{,.desc}:
    + [ADB] Add HTTPS to the list of recognised protocols for URLs in manpages
      which should not cause manpage-has-errors-from-man to be emitted.
      (Closes: #543641)
    + [RA] Document the MANWIDTH variable when running man by hand to
      check for man page errors.  (Closes: #542584)
    + [RA] Adjust the exclusion of pod2man's .IX macro for the new
      output format from groff.
  * checks/menu-format{,.desc}:
    + [RA] Add a pointer to desktop-file-validate in many of the tag
      descriptions for desktop tags.  (Closes: #539403)
    + [RA] Mention in the description of desktop-entry-invalid-category
      that whitespace isn't allowed and categories are case-sensitive.
    + [RA] Add a new check for desktop entries that are limited to
      multiple environments.  NotShowIn was probably intended instead.
      Thanks, Andrew Lee.  (Closes: #537737)
  * checks/rules:
    + [RA] Anchor the dh_quilt regex to not trigger on commented-out
      calls.  Patch from Cesare Tirabassi.  (Closes: #543259)
    + [RA] When checking for out-of-order debhelper calls, take into
      account the -p flag and only check within each package.  Thanks,
      Michael Koch.  (Closes: #545219)
  * checks/scripts{,.desc}:
    + [RA] Don't warn about missing dependencies on programs if the
      program invocation is followed by || true.  Thanks, Aurelien Jarno.
      (Closes: #541372)
    + [RA] Permit diversion of a man page as long as the package is
      installing a man page by the same name in the same numeric section,
      even if the specific section is different.  Thanks, Niko Tyni.
      (Closes: #543781)
    + [RA] Check example scripts for most of the same issues as scripts
      elsewhere in the package, but report problems with a different set
      of tags with info or pedantic severity.  Patch from Damyan Ivanov.
      (Closes: #543873)
    + [RA] Add seed.  Thanks, Josselin Mouette.  (Closes: #543779)
    + [RA] Add pike 7.8.  Thanks, Magnus Holmgren.  (Closes: #543262)
  * checks/shared-libs:
    + [RA] Allow ld.so to be executable.  (Closes: #538716)

* data/fields/essential:
    + [RA] New list of essential packages.
    + [RA] Add dash.  (Closes: #541744)
    + [RA] Add diffutils and remove diff.  (Closes: #544277)
  * data/fields/no-build-depends:
    + [RA] New list of packages that should not be used as build
      dependencies, currently containing only java-propose-classpath.
      Thanks, Josh Triplett.
  * data/files/fonts:
    + [RA] Refresh and include Type 1 fonts and packages as well.

* lib/Lintian/Collect/Binary.pm:
    + [RA] Stop looking for the special OCaml symbol.  OCaml custom
      binaries are now detected another way.  Patch from Stéphane Glondu.
  * lib/Lintian/Schedule.pm:
    + [RA] If declining to add a duplicate file to the Lintian check
      schedule, return success to avoid a spurious error message.  Patch
      from Raphael Geissert.  (Closes: #542072)
  * lib/Spelling.pm:
    + [RA] Add spelling correction for "aplication."  (Closes: #546404)

* private/refresh-fonts-data:
    + [RA] To match what checks/files looks for, also look for *.pfb files
      and packages starting with t1-.

* reporting/{html_reports,templates/clean.tmpl}:
    + [JP] Don't link to full report if the maintainer only has clean
      packages. (Closes: #545781)

-- Benjamin Drung <bdrung@ubuntu.com>   Tue, 22 Sep 2009 11:20:08 +0200

Changed in lintian (Ubuntu):
status:	Confirmed → Fix Released

Bug Watch Updater (bug-watch-updater) on 2009-10-07

Changed in lintian (Debian):
status:	Unknown → Won't Fix

Bug Watch Updater (bug-watch-updater) on 2011-08-16

Changed in lintian (Debian):
status:	Won't Fix → Fix Released

Affects		Status	Importance	Assigned to	Milestone
	lintian (Debian)	Fix Released	Unknown	debbugs #536434
	lintian (Ubuntu)	Fix Released	Undecided	Unassigned

Ubuntu
lintian package

Please disable "debian-changelog-file-is-a-symlink" warning

Bug Description

Other bug subscribers

Remote bug watches

Ubuntulintian package

Please disable "debian-changelog-file-is-a-symlink" warning

Bug Description

Other bug subscribers

Remote bug watches

Ubuntu
lintian package