haskell/haskell-ide-engine

Suggstion: ship with profiling disabled via stack-8.x.x.yaml

Open

#1.248 aberto em 8 de mai. de 2019

Ver no GitHub
 (5 comments) (0 reactions) (0 assignees)Haskell (205 forks)batch import
Well Defined Task - Awaiting PRbuildgardening 🌱good first issue

Métricas do repositório

Stars
 (2.358 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Related to #1214, which reports that building with profiling on under stack doesn't work.

Some developers may have profiling on by default with stack (i.e. library-profiling: true and executable-profiling: true in ~/.stack/config.yaml). I do this because I want profiling on more often than I want it off, and because it takes a long time to switch between profiling and non-profiling builds with stack. This configuration caused building HIE to fail and it took quite a while to figure out why.

As long as profiling doesn't work out of the box with stack, might it make sense ship with profiling explicitly disabled in stack-8.x.x.yaml?

build:
  library-profiling: false
  executable-profiling: false

Guia do colaborador

Direção de pesquisa
Adicione library profiling e executable profiling definidos como false no arquivo stack 8.x.x.yaml para substituir as configurações globais de criação de perfil.
Pilha de tecnologia
None
Domain
build system
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitStack
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80