[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#773300: Improve glibc bootstrap



On Tue, Dec 16, 2014 at 11:39:40PM +0800, YunQiang Su wrote:
> Hi, the attached patch can improve bootstrapping of glibc.

Partially, this seems to be a duplicate of #766877. Maybe these should
be merged?

> It produces the similiar stage1 glibc
> (libc6/libc6-dev and multilib version of them),
> at the same time, the dependencies of them are also correct.

The documentation and rationale of this patch are scarce. I have a few
comments on individual hunks though.

diff -Nru glibc-2.19/debian/rules glibc-2.19/debian/rules
--- glibc-2.19/debian/rules	2014-10-17 07:43:19.000000000 +0000
+++ glibc-2.19/debian/rules	2014-12-10 23:16:28.000000000 +0000
@@ -143,8 +143,12 @@
 endif
 endif
 
+ifeq ($(DEB_STAGE),stage2)
+  DEB_BUILD_PROFILES+=stage2
+endif
+
 ifneq ($(filter stage1,$(DEB_BUILD_PROFILES)),)
-  DEB_ARCH_REGULAR_PACKAGES = $(libc)-dev
+  DEB_ARCH_REGULAR_PACKAGES = $(libc)-dev $(libc)
   DEB_INDEP_REGULAR_PACKAGES = 
   DEB_UDEB_PACKAGES = 
 else

I have no clue how one would build the libc in stage1. The gcc stage1
does not provide the means for doing so. If anything those packages
would be empty. I.e. this seems rather wrong to me.

diff -Nru glibc-2.19/debian/sysdeps/linux.mk glibc-2.19/debian/sysdeps/linux.mk
--- glibc-2.19/debian/sysdeps/linux.mk	2014-07-16 18:43:31.000000000 +0000
+++ glibc-2.19/debian/sysdeps/linux.mk	2014-12-10 23:11:05.000000000 +0000
@@ -16,11 +16,7 @@
 endif
 
 ifndef LINUX_SOURCE
-  ifeq ($(DEB_HOST_GNU_TYPE),$(DEB_BUILD_GNU_TYPE))
-    LINUX_HEADERS := /usr/include
-  else
-    LINUX_HEADERS := /usr/$(DEB_HOST_GNU_TYPE)/include
-  endif
+  LINUX_HEADERS := /usr/include
   LINUX_ARCH_HEADERS := /usr/include/$(DEB_HOST_MULTIARCH)
 else
   LINUX_HEADERS := $(LINUX_SOURCE)/include

This breaks the supported cross build method.

diff -Nru glibc-2.19/debian/sysdeps/mips64.mk glibc-2.19/debian/sysdeps/mips64.mk
--- glibc-2.19/debian/sysdeps/mips64.mk	2014-10-17 07:43:19.000000000 +0000
+++ glibc-2.19/debian/sysdeps/mips64.mk	2014-12-11 03:50:09.000000000 +0000
@@ -59,5 +59,5 @@
 # create a symlink for the 32 bit dynamic linker in /lib
 define libc6-mips32_extra_pkg_install
 mkdir -p debian/libc6-mips32/lib
-ln -sf /libo32/ld.so.1 debian/libc6-mips32/lib
+ln -sf ../libo32/ld.so.1 debian/libc6-mips32/lib
 endef

This violates a should in Debian policy section 10.5.

Helmut


Reply to: