Project

General

Profile

Actions

Bug #9845

closed

make clobber in uts/intel does attempt to remove debug64/

Added by Toomas Soome about 5 years ago. Updated about 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
kernel
Start date:
2018-09-17
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:
External Bug:

Description

For some time I have been bugged by message in nightly.log:

/usr/bin/rm -f debug64/   ../intel/lint-libs/debug64/llib-l.ln /code/illumos-gat
e/usr/src/uts/common/os/priv_const.c Nothing_to_remove
rm: debug64/ is a directory
*** Error code 2 (ignored)

It is produced by make running in uts/intel, and specifically by clobber.targ target, apparently the CLOBBERFILES is set to list "$(BINARY) ..." and the BINARY itself is set $(OBJ_DIR)/$(MODULE)

However, we do not have MODULE set in uts/intel, so the BINARY is set to value "$(OBJ_DIR)/" - which does not make much sense here too. To reduce the build noise, just make sure BINARY is unset.

tsoome@beastie:/code/illumos-gate/usr/src/uts/intel$ dmake clobber.targ
/usr/bin/rm -f    ../intel/lint-libs/debug64/llib-l.ln /code/illumos-gate/usr/src/uts/common/os/priv_const.c Nothing_to_remove
Actions #1

Updated by Toomas Soome about 5 years ago

  • Subject changed from uts/intel does arrempt to remove debug64/ to uts/intel does attempt to remove debug64/
Actions #2

Updated by Toomas Soome about 5 years ago

  • Subject changed from uts/intel does attempt to remove debug64/ to make clobber in uts/intel does attempt to remove debug64/
Actions #3

Updated by Electric Monk about 5 years ago

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

git commit c057d312c6f715bb3aeadb653466e7046f26c4af

commit  c057d312c6f715bb3aeadb653466e7046f26c4af
Author: Toomas Soome <tsoome@me.com>
Date:   2018-09-21T14:39:30.000Z

    9845 make clobber in uts/intel does attempt to remove debug64/
    Reviewed by: Robert Mustacchi <rm@joyent.com>
    Approved by: Dan McDonald <danmcd@joyent.com>

Actions

Also available in: Atom PDF