mapbox / mapbox/node-cpp-skel

Make default release mode build a "profiling" build

Offen
#117 0 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

For the ability to profile our code with perf (and to generate the most readable flamegraphs) it is important to:

  • Compile with -fno-omit-frame-pointer (unless the compiler already defaults to using this flag?)
  • Compile with some debug symbols (unless you don't need detailed stack traces of your node module)

Doing this would move node-cpp-skel to doing a "profiling" build by default: https://github.com/mapbox/cpp/blob/master/glossary.md#profiling-build.

We can accomplish this by applying something like https://github.com/mapnik/node-mapnik/pull/804 to node-cpp-skel.

refs https://twitter.com/brendangregg/status/472869470675795968

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 die standardmäßige Release-Build-Konfiguration des Repositorys und vergleiche anschließend den gewünschten Ansatz mit node-mapnik PR #804. Bestätige, dass der Standard-Build -fno-omit-frame-pointer verwendet und Debug-Symbole enthält, und überprüfe, dass das resultierende Node-Modul weiterhin gebaut werden kann und für perf-Flamegraphs geeignet ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

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

Neue Issues direkt in Ihr Postfach

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