From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with archive (Exim 4.43) id 1KbYyo-0007As-EC for mharc-grub-devel@gnu.org; Fri, 05 Sep 2008 06:55:54 -0400 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1KbYyl-00077J-S5 for grub-devel@gnu.org; Fri, 05 Sep 2008 06:55:51 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1KbYyk-00073z-C4 for grub-devel@gnu.org; Fri, 05 Sep 2008 06:55:50 -0400 Received: from [199.232.76.173] (port=35503 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1KbYyj-000720-MS for grub-devel@gnu.org; Fri, 05 Sep 2008 06:55:49 -0400 Received: from aybabtu.com ([69.60.117.155]:34741) by monty-python.gnu.org with esmtps (TLS-1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1KbYyj-0003Gz-Pv for grub-devel@gnu.org; Fri, 05 Sep 2008 06:55:50 -0400 Received: from [192.168.10.10] (helo=thorin) by aybabtu.com with esmtps (TLS-1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.69) (envelope-from ) id 1KbYoE-00026L-60 for grub-devel@gnu.org; Fri, 05 Sep 2008 12:45:01 +0200 Received: from rmh by thorin with local (Exim 4.63) (envelope-from ) id 1KbYx1-0006z1-Kp for grub-devel@gnu.org; Fri, 05 Sep 2008 12:54:03 +0200 Date: Fri, 5 Sep 2008 12:54:03 +0200 From: Robert Millan To: The development of GRUB 2 Message-ID: <20080905105403.GA26724@thorin> References: <20080803192839.GA12627@thorin> <20080803193027.GA12766@thorin> <1217825941.10989.26.camel@dv> <4897FA04.1060001@nic.fi> <1217954986.2722.4.camel@dv> <20080805200715.GA1113@thorin> <1217970509.2908.3.camel@dv> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="LZvS9be/3tNcYl/X" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1217970509.2908.3.camel@dv> Organization: free as in freedom X-Message-Flag: Worried about Outlook viruses? Switch to Thunderbird! www.mozilla.com/thunderbird X-Debbugs-No-Ack: true User-Agent: Mutt/1.5.13 (2006-08-11) X-detected-kernel: by monty-python.gnu.org: Genre and OS details not recognized. Subject: [PATCH] Move cpuid.mod to conf/i386.rmk (Re: RFC: conf/i386.rmk) X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: The development of GRUB 2 List-Id: The development of GRUB 2 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 05 Sep 2008 10:55:52 -0000 --LZvS9be/3tNcYl/X Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit On Tue, Aug 05, 2008 at 05:08:29PM -0400, Pavel Roskin wrote: > On Tue, 2008-08-05 at 22:07 +0200, Robert Millan wrote: > > On Tue, Aug 05, 2008 at 12:49:46PM -0400, Pavel Roskin wrote: > > > On Tue, 2008-08-05 at 09:58 +0300, Vesa Jääskeläinen wrote: > > > > > > > > I tried moving more stuff to common.rmk many times but gave up every > > > > > time. One of the reasons is that the sparc64 support is very > > > > > out-of-date and doesn't use common.rmk at all. I cannot even test it > > > > > (well, I haven't tries hard). > > > > > > > > If sparc64 support is out of date and the maintainer is nowhere to be > > > > seen then I think you can put that support to graveyard until someone > > > > comes up to update it. Putting all common stuff to every platform to > > > > common.rmk is a good way and the only way (in my opinion) to go forward. > > > > > > OK, then there is an issue that it doesn't work if done naively. > > > > What is the problem exactly? > > Take all text in conf/i386-pc.rmk from "For grub-emu" to "Scripts" and > move it to the end of conf/common.rmk. Run autogen.sh, configure > --enable-grub-emu and make. You'll get: > > make: *** No rule to make target `grub-emu', needed by `all-local'. > Stop. Okay then. Since it appears this kind of changes are delicate, I propose doing them granularly. Here's my first patch, that moves cpuid.mod and sets the base for moving more stuff later (I've tested the resulting cpuid.mod is built correctly). -- Robert Millan The DRM opt-in fallacy: "Your data belongs to us. We will decide when (and how) you may access your data; but nobody's threatening your freedom: we still allow you to remove your data and not access it at all." --LZvS9be/3tNcYl/X Content-Type: text/x-diff; charset=us-ascii Content-Disposition: attachment; filename="i386.diff" 2008-09-05 Robert Millan * conf/i386.rmk: New file. Provides declaration for building `cpuid.mod'. * conf/i386-pc.rmk (pkglib_MODULES): Remove `cpuid.mod'. (cpuid_mod_SOURCES, cpuid_mod_CFLAGS, cpuid_mod_LDFLAGS): Remove variables. Include `conf/i386.mk'. * conf/i386-efi.rmk: Likewise. * conf/x86_64-efi.rmk: Likewise. * conf/i386-coreboot.rmk: Likewise. * conf/i386-ieee1275.rmk: Likewise. Index: conf/i386.rmk =================================================================== --- conf/i386.rmk (revision 0) +++ conf/i386.rmk (revision 0) @@ -0,0 +1,8 @@ +# -*- makefile -*- + +pkglib_MODULES += cpuid.mod + +# For cpuid.mod. +cpuid_mod_SOURCES = commands/i386/cpuid.c +cpuid_mod_CFLAGS = $(COMMON_CFLAGS) +cpuid_mod_LDFLAGS = $(COMMON_LDFLAGS) Index: conf/i386-pc.rmk =================================================================== --- conf/i386-pc.rmk (revision 1851) +++ conf/i386-pc.rmk (working copy) @@ -163,7 +163,7 @@ pkglib_MODULES = biosdisk.mod _chain.mod _linux.mod linux.mod normal.mod \ _multiboot.mod chain.mod multiboot.mod reboot.mod halt.mod \ vbe.mod vbetest.mod vbeinfo.mod video.mod gfxterm.mod \ - videotest.mod play.mod bitmap.mod tga.mod cpuid.mod serial.mod \ + videotest.mod play.mod bitmap.mod tga.mod serial.mod \ ata.mod vga.mod memdisk.mod jpeg.mod png.mod pci.mod lspci.mod \ aout.mod _bsd.mod bsd.mod pxe.mod pxecmd.mod datetime.mod date.mod \ datehook.mod @@ -278,11 +278,6 @@ tga_mod_CFLAGS = $(COMMON_CFLAGS) tga_mod_LDFLAGS = $(COMMON_LDFLAGS) -# For cpuid.mod. -cpuid_mod_SOURCES = commands/i386/cpuid.c -cpuid_mod_CFLAGS = $(COMMON_CFLAGS) -cpuid_mod_LDFLAGS = $(COMMON_LDFLAGS) - # For ata.mod. ata_mod_SOURCES = disk/ata.c ata_mod_CFLAGS = $(COMMON_CFLAGS) @@ -358,4 +353,5 @@ datehook_mod_CFLAGS = $(COMMON_CFLAGS) datehook_mod_LDFLAGS = $(COMMON_LDFLAGS) +include $(srcdir)/conf/i386.mk include $(srcdir)/conf/common.mk Index: conf/i386-efi.rmk =================================================================== --- conf/i386-efi.rmk (revision 1851) +++ conf/i386-efi.rmk (working copy) @@ -76,7 +76,7 @@ # Modules. pkglib_MODULES = kernel.mod normal.mod _chain.mod chain.mod appleldr.mod \ - _linux.mod linux.mod cpuid.mod halt.mod reboot.mod pci.mod lspci.mod \ + _linux.mod linux.mod halt.mod reboot.mod pci.mod lspci.mod \ datetime.mod date.mod datehook.mod # For kernel.mod. @@ -144,11 +144,6 @@ linux_mod_CFLAGS = $(COMMON_CFLAGS) linux_mod_LDFLAGS = $(COMMON_LDFLAGS) -# For cpuid.mod. -cpuid_mod_SOURCES = commands/i386/cpuid.c -cpuid_mod_CFLAGS = $(COMMON_CFLAGS) -cpuid_mod_LDFLAGS = $(COMMON_LDFLAGS) - # For halt.mod. halt_mod_SOURCES = commands/halt.c halt_mod_CFLAGS = $(COMMON_CFLAGS) @@ -184,4 +179,5 @@ datehook_mod_CFLAGS = $(COMMON_CFLAGS) datehook_mod_LDFLAGS = $(COMMON_LDFLAGS) +include $(srcdir)/conf/i386.mk include $(srcdir)/conf/common.mk Index: conf/x86_64-efi.rmk =================================================================== --- conf/x86_64-efi.rmk (revision 1851) +++ conf/x86_64-efi.rmk (working copy) @@ -78,7 +78,7 @@ # Modules. pkglib_MODULES = kernel.mod normal.mod _chain.mod chain.mod appleldr.mod \ - cpuid.mod halt.mod reboot.mod _linux.mod linux.mod pci.mod lspci.mod \ + halt.mod reboot.mod _linux.mod linux.mod pci.mod lspci.mod \ datetime.mod date.mod datehook.mod # For kernel.mod. @@ -146,11 +146,6 @@ linux_mod_CFLAGS = $(COMMON_CFLAGS) linux_mod_LDFLAGS = $(COMMON_LDFLAGS) -# For cpuid.mod. -cpuid_mod_SOURCES = commands/i386/cpuid.c -cpuid_mod_CFLAGS = $(COMMON_CFLAGS) -cpuid_mod_LDFLAGS = $(COMMON_LDFLAGS) - # For halt.mod. halt_mod_SOURCES = commands/halt.c halt_mod_CFLAGS = $(COMMON_CFLAGS) @@ -186,4 +181,5 @@ datehook_mod_CFLAGS = $(COMMON_CFLAGS) datehook_mod_LDFLAGS = $(COMMON_LDFLAGS) +include $(srcdir)/conf/i386.mk include $(srcdir)/conf/common.mk Index: conf/i386-coreboot.rmk =================================================================== --- conf/i386-coreboot.rmk (revision 1851) +++ conf/i386-coreboot.rmk (working copy) @@ -94,7 +94,7 @@ # Modules. pkglib_MODULES = _linux.mod linux.mod normal.mod \ _multiboot.mod multiboot.mod aout.mod \ - play.mod cpuid.mod serial.mod ata.mod \ + play.mod serial.mod ata.mod \ memdisk.mod pci.mod lspci.mod reboot.mod \ halt.mod datetime.mod date.mod datehook.mod @@ -156,11 +156,6 @@ play_mod_CFLAGS = $(COMMON_CFLAGS) play_mod_LDFLAGS = $(COMMON_LDFLAGS) -# For cpuid.mod. -cpuid_mod_SOURCES = commands/i386/cpuid.c -cpuid_mod_CFLAGS = $(COMMON_CFLAGS) -cpuid_mod_LDFLAGS = $(COMMON_LDFLAGS) - # For ata.mod. ata_mod_SOURCES = disk/ata.c ata_mod_CFLAGS = $(COMMON_CFLAGS) @@ -196,4 +191,5 @@ datehook_mod_CFLAGS = $(COMMON_CFLAGS) datehook_mod_LDFLAGS = $(COMMON_LDFLAGS) +include $(srcdir)/conf/i386.mk include $(srcdir)/conf/common.mk Index: conf/i386-ieee1275.rmk =================================================================== --- conf/i386-ieee1275.rmk (revision 1851) +++ conf/i386-ieee1275.rmk (working copy) @@ -96,7 +96,7 @@ grub_install_SOURCES = util/ieee1275/grub-install.in # Modules. -pkglib_MODULES = normal.mod halt.mod reboot.mod suspend.mod cpuid.mod \ +pkglib_MODULES = normal.mod halt.mod reboot.mod suspend.mod \ multiboot.mod _multiboot.mod aout.mod serial.mod linux.mod \ _linux.mod nand.mod memdisk.mod pci.mod lspci.mod datetime.mod \ date.mod datehook.mod @@ -143,11 +143,6 @@ halt_mod_CFLAGS = $(COMMON_CFLAGS) halt_mod_LDFLAGS = $(COMMON_LDFLAGS) -# For cpuid.mod. -cpuid_mod_SOURCES = commands/i386/cpuid.c -cpuid_mod_CFLAGS = $(COMMON_CFLAGS) -cpuid_mod_LDFLAGS = $(COMMON_LDFLAGS) - # For serial.mod. serial_mod_SOURCES = term/i386/pc/serial.c serial_mod_CFLAGS = $(COMMON_CFLAGS) @@ -198,4 +193,5 @@ datehook_mod_CFLAGS = $(COMMON_CFLAGS) datehook_mod_LDFLAGS = $(COMMON_LDFLAGS) +include $(srcdir)/conf/i386.mk include $(srcdir)/conf/common.mk --LZvS9be/3tNcYl/X--