mesonbuild/meson

Add subdir argument to gnome.mkenums_simple() to match install_headers()

Aberta

#2.969 aberto em 24 de jan. de 2018

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (0 responsável)Python (1.465 forks)batch import
enhancementgnomegood first issuehelp wanted

Métricas do repositório

Stars
 (4.978 estrelas)
Métricas de merge de PR
 (Mesclagem média 139d 13h) (51 fundiu PRs em 30d)

Description

Currently, to install my public headers and generated enums.h, I have to do something like:

includedir = join_paths(prefix, get_option('includedir'))
libblah_include_subdir = join_paths(libblah_api_name, 'libblah')
libblah_enums = gnome.mkenums_simple('enums',
  sources: libblah_headers,
  install_header: true,
  install_dir: join_paths(includedir, libblah_include_subdir),
)

# Public library bits.
install_headers(libblah_headers,
  subdir: libblah_include_subdir,
)

It would be good if gnome.mkenums_simple() took a subdir argument like install_headers, so that dealing with includedir directly was not necessary.

Guia do colaborador

Direção de pesquisa
Encontre a implementação de `gnome.mkenums simple` no código fonte do Meson, provavelmente em `mesonbuild/modules/gnome.py` ou similar. Adicione um argumento de palavra chave `subdir`. Quando `install header` for verdadeiro, calcule o diretório de instalação concatenando o valor de `subdir` com o diretório de instalação padrão (ou o `install dir` fornecido). Garanta que a mudança seja consistente com a forma como `install headers` lida com `subdir`. Atualize a documentação e adicione testes unitários para a nova funcionalidade.
Pilha de tecnologia
python
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.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PythonMeson build system
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75