Microsoft/vscode

RPM %post scriplet deletes /usr/local/bin/code

Open

#236 132 ouverte le 14 déc. 2024

Voir sur GitHub
 (1 commentaire) (1 réaction) (2 assignés)TypeScript (10 221 forks)batch import
bughelp wantedinstall-updatelinuxpackagingrpm

Métriques du dépôt

Stars
 (74 848 stars)
Métriques de merge PR
 (Merge moyen 11h 43m) (1 000 PRs mergées en 30 j)

Description

Does this issue occur when all extensions are disabled?: Yes

  • VS Code Version: 1.96.0 (RPM from the Microsoft repo)
  • OS Version: Fedora 41

The %post scriptlet in the RPM spec file deletes /usr/local/bin/code even if it doesn't belong to a previous vscode installation. This is very irritating because it's the easiest way to append arbitrary command line arguments such as --ozone-platform=wayland system-wide. Please either add some validation before deleting it, or just stop deleting it altogether.

https://github.com/microsoft/vscode/blob/1.96.0/resources/linux/rpm/code.spec.template#L48-L51

Steps to Reproduce:

  1. Install a script at /usr/local/bin/code which launches /usr/bin/code with extra arguments
  2. Reinstall the RPM file provided by Microsoft
  3. /usr/local/bin/code is now deleted

Guide contributeur

Direction de recherche
Vérifiez le scriptlet %post dans le modèle de spec RPM et ajoutez une vérification avant de supprimer /usr/local/bin/code pour vous assurer qu'il s'agit d'un lien symbolique créé par le package VS Code lui même, et non d'un script créé par l'utilisateur.
Stack technique
shell
Domaine
build system
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
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
Shell scriptingRPM packaging basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70