Project

General

Profile

Actions

Bug #13270

closed

cmd/spell native build incomplete

Added by Robert Mustacchi over 2 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Category:
cmd - userland programs
Start date:
Due date:
% Done:

100%

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

Description

The cmd/spell build has a native program and while it make sure to set up CC and CPPFLAGS for the native build, it doesn't do the same for CFLAGS and LDLIBS. This should be fixed to always use the native paths.


Related issues

Blocks illumos gate - Feature #13274: enable -fstack-protector-strong by default in user landClosedRobert Mustacchi

Actions
Actions #1

Updated by Robert Mustacchi over 2 years ago

  • Blocks Feature #13274: enable -fstack-protector-strong by default in user land added
Actions #2

Updated by Electric Monk over 2 years ago

  • Gerrit CR set to 1031
Actions #3

Updated by Robert Mustacchi over 2 years ago

I tested this primarily through wsdiff and verified that there were no changes induced by this. I also verified the build on systems with and without the stack protector work that led to me discovering this.

Actions #4

Updated by Electric Monk over 2 years ago

  • Status changed from New to Closed
  • % Done changed from 90 to 100

git commit dd49d7b32f2281b4a30555d0ef9bf80720bbb67f

commit  dd49d7b32f2281b4a30555d0ef9bf80720bbb67f
Author: Robert Mustacchi <rm@fingolfin.org>
Date:   2020-11-04T15:32:32.000Z

    13270 cmd/spell native build incomplete
    Reviewed by: Richard Lowe <richlowe@richlowe.net>
    Reviewed by: Patrick Mooney <pmooney@pfmooney.com>
    Reviewed by: Toomas Soome <tsoome@me.com>
    Approved by: Gordon Ross <gordon.w.ross@gmail.com>

Actions

Also available in: Atom PDF