ocaml/dune

Preprocess using both cppo and PPXs

Offen

#171 geöffnet am 27.06.2017

Auf GitHub ansehenContributor Guide
 (14 Kommentare) (5 Reaktionen) (0 zugewiesene Personen)OCaml (478 Forks)github user discovery
help wantedpreprocessing

Repository-Metriken

Stars
 (1.882 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 1T 20h) (345 gemergte PRs in 30 T)

Beschreibung

From https://github.com/ocsigen/lwt/issues/411#issuecomment-311214821:

it is possible to both apply a PPX and preprocess with cppo, but it requires the [separate] cppo rule stanza, and so presumably a source file with a name like foo.cppo.ml. It would be nice if one could both apply cppo and PPXs to source named foo.ml.

This is not a high priority, but it would make files like lwt_unix.cppo.ml, where we both want to use cppo, and apply Bisect_ppx, ever so slightly more maintainable.

I'm basically thinking of being able to have either multiple preprocess stanzas, or list both pps and action in one preprocess stanza. There may, of course, be other, better ways.

Contributor Guide

Research-Richtung
Untersuche die aktuelle Implementierung des preprocess Blocks in dune. Sieh dir an, wie cppo und PPX getrennt behandelt werden. Identifiziere den Codepfad zum Parsen des preprocess Blocks und überlege, wie beide kombiniert werden können (z.B. durch Akzeptieren einer Liste oder Verwendung einer neuen Syntax). Sieh dir vorhandene Tests und Dokumentation als Leitfaden an.
Tech Stack
Keine
Domain
build system
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
OCamldune build system
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50