mapbox / mapbox/node-cpp-skel

Avoiding hardcoded build options

Offen
#133 3 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
C++
Sterne
74
Forks
9
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Currently in order to build on Ubuntu Linux (18.04) using either system g++7 or source compiled clang++7 I had to apply following patch:

CXX=/usr/bin/g++ make
diff --git a/common.gypi b/common.gypi
index 803f217..6ae5c84 100644
--- a/common.gypi
+++ b/common.gypi
@@ -38,11 +38,8 @@
           'NDEBUG'
         ],
         'cflags': [
-         '-flto'
         ],
         'ldflags': [
-         '-flto',
-         '-fuse-ld=<(module_root_dir)/mason_packages/.link/bin/ld'
         ],
         'xcode_settings': {
           'OTHER_CPLUSPLUSFLAGS!': [

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Untersuche common.gypi und den im Issue gezeigten Build-Aufruf und reproduziere den Ubuntu-Build mit systemseitig installiertem g++7 oder aus dem Quellcode gebautem clang++7. Als abgeschlossen gilt die Aufgabe, wenn das Projekt gebaut werden kann, ohne die fest codierten -flto-Flags und den Mason-Linkerpfad aus common.gypi manuell zu entfernen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp, node.js
Bereich
build-system
Issue-Typ
Refactoring
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.