fix: preserve package lists and detect profile generation failures - #10
Merged
Merged
Conversation
Handle zero-byte removal files by matching awk inputs by filename. Propagate real git commit and diff failures from Make Profiles. Validate upstream KDE sections before appending, while allowing the legitimately empty Misc section. Add 46 regression cases and a read-only push/PR test workflow. Leave generated profiles untouched.
Member
Author
Validação confirmada no GitHub ActionsO workflow Test profile tooling do evento
Execução: https://github.com/biglinux/iso-profiles/actions/runs/34146961247 Os logs foram consultados para confirmar os números acima. Isso complementa a validação local descrita no PR, mas não substitui uma construção e inicialização reais da ISO. Nenhum merge foi realizado. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Objetivo
Corrigir falhas identificadas na revisão do motor de construção e da geração de perfis, acrescentando testes de regressão e execução automática da suíte.
Alterações
NR==FNRporFILENAME == ARGV[1]emapply_profile_removals(). Um arquivo*-removede zero bytes deixa de apagar silenciosamente a listaPackages-*. Preserva o tratamento de links simbólicos e o staging para a limpeza no chroot.git commitemmake-profiles.yml. Ausência de alterações continua sendo sucesso; falhas de commit e erros reais degit diffpassam a interromper a etapa. Não depende de mensagens do Git nem de seu idioma.## Miscvazio como caso válido, conforme o perfil atualmente versionado. Uma seção ausente ou essencial vazia não deixa saída parcialmente modificada..github/workflows/test.ymlparapush,pull_requeste execução manual. Verifica sintaxe Bash e executapytest -q -ra build-iso/tests/, com permissão somente de leitura e sem persistir credenciais no checkout.Testes
São 46 cenários novos: 24 para listas vazias/comentários/espaços, arquivos regulares e links simbólicos nas quatro categorias; 6 para a etapa de commit em repositórios Git temporários; 16 para a extração das seções KDE.
Os testes das correções foram executados localmente. Após ajustar a compatibilidade com
## Miscvazio, os 16 testes KDE foram repetidos e passaram. Também foi verificada a sintaxe Bash dos scripts alterados.Limites da validação: a suíte preexistente completa não foi executada integralmente no ambiente local; o novo workflow foi incluído para executá-la no GitHub. Não foi construída nem inicializada uma ISO. Os testes de seções usam fixtures e o formato do perfil versionado, não constituem uma construção integrada contra o Manjaro atual.
Escopo e preservação
Base:
68ace70b34ccf95bc4a2e90d6fb902e5dbd477de.São 7 arquivos alterados/adicionados. Nenhum arquivo gerado em
biglinux/foi editado; a seleção de pacotes e a política de remoção por substring do gerador permanecem inalteradas. Não há merge automático nem alteração direta damain.