ocaml/dune

Preprocess using both cppo and PPXs

Ouverte

#171 ouverte le 27 juin 2017

Voir sur GitHubGuide contributeur
 (14 commentaires) (5 réactions) (0 personne assignée)OCaml (478 forks)github user discovery
help wantedpreprocessing

Métriques du dépôt

Stars
 (1 882 étoiles)
Métriques de merge PR
 (Merge moyen 1j 20h) (345 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez l'implémentation actuelle de la strophe preprocess dans dune. Regardez comment cppo et PPX sont traités séparément. Identifiez le chemin du code pour analyser la strophe preprocess et considérez comment permettre de combiner les deux (par exemple, en acceptant une liste ou en utilisant une nouvelle syntaxe). Consultez les tests et la documentation existants pour vous guider.
Stack technique
Aucun
Domaine
build system
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
OCamldune build system
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50