Project

General

Profile

Bug #9544

loader: i386/Makefile should have target all before other targets

Added by Toomas Soome over 1 year ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Toomas Soome
Category:
bootloader
Start date:
2018-05-21
Due date:
% Done:

100%

Estimated time:
Difficulty:
Bite-size
Tags:

Description

i386/Makefile has targets in bad order, causing "default" build to not build all:

tsoome@beastie:/code/illumos-gate/usr/src/boot/sys/boot/i386$ dmake
/code/illumos-gate/usr/src/boot/sys/boot/i386/loader
/code/illumos-gate/usr/src/boot/sys/boot/i386/pxeldr
`pxeboot' is up to date.
tsoome@beastie:/code/illumos-gate/usr/src/boot/sys/boot/i386$

And with fixed order:

tsoome@beastie:/code/illumos-gate/usr/src/boot/sys/boot/i386$ dmake
/code/illumos-gate/usr/src/boot/sys/boot/i386/pmbr
/code/illumos-gate/usr/src/boot/sys/boot/i386/btx
beastie --> 1 job
beastie --> 2 jobs
beastie --> 3 jobs
beastie --> Job output
/code/illumos-gate/usr/src/boot/sys/boot/i386/btx/btx
beastie --> Job output
/code/illumos-gate/usr/src/boot/sys/boot/i386/btx/btxldr
beastie --> Job output
/code/illumos-gate/usr/src/boot/sys/boot/i386/btx/lib
/code/illumos-gate/usr/src/boot/sys/boot/i386/cdboot
/code/illumos-gate/usr/src/boot/sys/boot/i386/gptzfsboot
/code/illumos-gate/usr/src/boot/sys/boot/i386/libi386
/code/illumos-gate/usr/src/boot/sys/boot/i386/loader
/code/illumos-gate/usr/src/boot/sys/boot/i386/pxeldr
tsoome@beastie:/code/illumos-gate/usr/src/boot/sys/boot/i386$

History

#1

Updated by Electric Monk over 1 year ago

  • % Done changed from 90 to 100
  • Status changed from In Progress to Closed

git commit 207bc6d049369709a154c840750e0a74a9ff0a76

commit  207bc6d049369709a154c840750e0a74a9ff0a76
Author: Toomas Soome <tsoome@me.com>
Date:   2018-06-02T14:48:51.000Z

    9544 loader: i386/Makefile should have target all before other targets
    Reviewed by: Andrew Stormont <andyjstormont@gmail.com>
    Reviewed by: Dan McDonald <danmcd@joyent.com>
    Approved by: Robert Mustacchi <rm@joyent.com>

Also available in: Atom PDF