linux-libc-dev: please include video/uvesafb.h

Bug #300803 reported by Evgeni Golov on 2008-11-21
10
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Medium
Andy Whitcroft
Intrepid
Medium
Andy Whitcroft

Bug Description

Hi,

currently Ubuntu does not ship /usr/include/video/uvesafb.h in linux-libc-dev (it does in various linux-headers-* but that does not help while building packages needing that file), as a result of this, v86d has to ship an own (maybe outdated) copy of that file. Could you please ship it? That would make the diff between Debian and Ubuntu v86d smaller (actually zero) and make our life easier.

Regards
Evgeni Golov
Debian v86d Maintainer

Changed in linux:
assignee: nobody → ubuntu-kernel-team
importance: Undecided → Medium
status: New → Triaged
Andy Whitcroft (apw) on 2008-12-05
Changed in linux:
assignee: ubuntu-kernel-team → apw
status: Triaged → In Progress
Andy Whitcroft (apw) wrote :

It seems that there are a number of other directories of userspace header files generated by the kernel install which we do not include. It is probabally appropriate that we include all of these in our package. I will prepare a patch and post it for consideration.

Tim Gardner (timg-tpi) on 2008-12-05
Changed in linux:
assignee: nobody → apw
importance: Undecided → Medium
milestone: none → intrepid-updates
status: New → In Progress
Andy Whitcroft (apw) wrote :

Pushed the patch to both the Jaunty and Intrepid trees. Should be included in the next updates.

Changed in linux:
status: In Progress → Fix Committed
status: In Progress → Fix Committed
Andy Whitcroft (apw) wrote :

SRU Justification:

Impact: Standard header files are missing on a standard install

Fix Description: includes all exported kernel userspace headers in linux-libc-dev

Patch: http://kernel.ubuntu.com/git?p=ubuntu/ubuntu-intrepid.git;a=commit;h=481456272646803dfb557e82fac6a4ada2b002f8

Risks: low, these files are already installed on Debian

TEST CASE: see bug report

Steve Langasek (vorlon) wrote :

Accepted into intrepid-proposed, please test and give feedback here. Please see https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Loïc Minier (lool) wrote :

Andy this caused a regression with libdrm-dev:
dpkg : erreur de traitement de /var/cache/apt/archives/linux-libc-dev_2.6.27-11.21_amd64.deb (--unpack) :
 tentative de remplacement de « /usr/include/drm/drm_sarea.h », qui appartient aussi au paquet libdrm-dev

I think you shouldn't touch something as fundamental as the linux-libc-dev headers in a released distro; we rely on being able to build packages for security updates and this change might break random packages without us noticing until we need to build them.

Colin Watson (cjwatson) wrote :

I agree - we'd really want a full test rebuild at some point to validate this change. Including it in Jaunty alone should be sufficient, as we won't be doing anything about the v86d Debian->Ubuntu delta in Intrepid anyway.

On Fri, Dec 19, 2008 at 10:39:42AM -0000, Colin Watson wrote:
> I agree - we'd really want a full test rebuild at some point to validate
> this change. Including it in Jaunty alone should be sufficient, as we
> won't be doing anything about the v86d Debian->Ubuntu delta in Intrepid
> anyway.

Yes, this has not worked out in quite the way expected. This was perceived
to be low risk as we were moving into line with the Debian upstream way of
handling these headers. That seems to have been an erroneous assumption.
In that the current Debian upstream state is also broken.

The drm header collision was spotted recently and although we proposed
removing the headers for drm from the change, it was decided that
updating drm to Replaces: the files was a smaller change and would
prevent the need to rebuild the kernel. This doesn not seem to have
worked out right either.

Given the mess that has resulted, it does seem premature to have this
change in Intrepid for sure. Will coordinate getting this sorted out.

Andy Whitcroft (apw) wrote :

Ok we have had to revert this change in Intrepid as there are a number of complex interactions with other packages including libdrm-dev. This is triggering install failures directly and build failures in a couple of key packages. Therefore this is not suitable for a stable series. The change is already in Jaunty and we will work the issues there.

Changed in linux:
status: Fix Committed → Won't Fix
Colin Watson (cjwatson) wrote :

I've accepted the reversion (linux 2.6.27-11.22). Thanks.

Martin Pitt (pitti) on 2009-01-07
Changed in linux:
milestone: intrepid-updates → none
Launchpad Janitor (janitor) wrote :
Download full text (16.6 KiB)

This bug was fixed in the package linux - 2.6.27-11.25

---------------
linux (2.6.27-11.25) intrepid-proposed; urgency=low

  [ Jeff Layton ]

  * SAUCE: cifs: make sure we allocate enough storage for socket address
    - LP: #318565

linux (2.6.27-11.24) intrepid-proposed; urgency=low

  [ Stefan Bader ]

  * Revert "SAUCE: don't use buggy _BCL/_BCM/_BQC for backlight control"
    - LP: #311716
  * SAUCE: acpi: Hack to enable video and vendor backlight implementations
    - LP: #311716
  * SAUCE: Force vendor backlight control on ThinkPad T61
    - LP: #311716

  [ Upstream Kernel Changes ]

  * Revert "thinkpad_acpi: fingers off backlight if video.ko is serving
    this functionality"
    - LP: #311716

linux (2.6.27-11.23) intrepid-proposed; urgency=low

  [ Andy Whitcroft ]

  * SAUCE: don't use buggy _BCL/_BCM/_BQC for backlight control
    - LP: #311716, #314119

  [ Jim Lieb ]

  * SAUCE: atl2: add tx bytes statistic
    - LP: #268622

  [ Upstream Kernel Changes ]

  * i915: Save/restore MCHBAR_RENDER_STANDBY on GM965/GM45
    - LP: #276943

linux (2.6.27-11.22) intrepid-proposed; urgency=low

  [ Andy Whitcroft ]

  * Revert "synchronise our linux-libc-dev with the kernel userspace
    headers". It was causing regressions.
    - LP: #300803

  [ Brian Rogers ]

  * SAUCE: Add support for MSI TV@nywhere Plus remote
    - LP: #281647

  [ Tim Gardner ]

  * SAUCE: Dell laptop digital mic does not work, PCI 1028:0271
    - LP: #309508

  [ Upstream Kernel Changes ]

  * Revert "sched_clock: prevent scd->clock from moving backwards"
  * AMD IOMMU: enable device isolation per default
  * bonding: fix miimon failure counter
  * x86 Fix VMI crash on boot in 2.6.28-rc8
  * libata: fix Seagate NCQ+FLUSH blacklist
  * e1000e: fix double release of mutex
  * can: Fix CAN_(EFF|RTR)_FLAG handling in can_filter
  * can: omit received RTR frames for single ID filter lists
  * iwlwifi: clean key table in iwl_clear_stations_table function
  * net: eliminate warning from NETIF_F_UFO on bridge
  * unicode table for cp437
  * console ASCII glyph 1:1 mapping
  * key: fix setkey(8) policy set breakage
  * firewire: fw-ohci: fix IOMMU resource exhaustion
  * ieee1394: add quirk fix for Freecom HDD
  * SUNRPC: Fix a performance regression in the RPC authentication code
  * b1isa: fix b1isa_exit() to really remove registered capi controllers
  * macfb: Do not overflow fb_fix_screeninfo.id
  * setup_per_zone_pages_min(): take zone->lock instead of zone->lru_lock
  * xilinx_hwicap: remove improper wording in license statement
  * Linux 2.6.27.10 except for "iwlagn: fix RX skb alignment". Besides causing
    an ABI bump it only applies to machines using > 4K page size (such as PowerPC).
    Pick this one up on the next ABI bumping upload.
    - LP: #309731

linux (2.6.27-11.21) intrepid-proposed; urgency=low

  [ Andy Whitcroft ]

  * synchronise our linux-libc-dev with the kernel userspace headers
    - LP: #300803

  [ Jim Lieb ]

  * SAUCE: [PATCH 1/1] USB: Unusual devs patch for Nokia 5610
    - LP: #287701

  [ Michael Krufky ]

  * sms1xxx: use new firmware for Hauppauge WinTV MiniStick
    - LP: #299671
  * sms1xxx: add autodetection support for Hauppa...

Changed in linux:
status: Won't Fix → Fix Released

Cleaning up tasks to reflect Fix Released in Jaunty but Won't Fix for Intrepid.

Changed in linux (Ubuntu):
status: Fix Committed → Fix Released
Changed in linux (Ubuntu Intrepid):
status: Fix Released → Won't Fix
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers