Microsoft/vscode

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

Open

#236.132 geöffnet am 14. Dez. 2024

Auf GitHub ansehen
 (1 Kommentar) (1 Reaktion) (2 zugewiesene Personen)TypeScript (10.221 Forks)batch import
bughelp wantedinstall-updatelinuxpackagingrpm

Repository-Metriken

Stars
 (74.848 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 11h 43m) (1.000 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Überprüfen Sie das %post Skriptlet in der RPM Spec Vorlage und fügen Sie vor dem Löschen von /usr/local/bin/code eine Überprüfung hinzu, um sicherzustellen, dass es sich um einen vom VS Code Paket selbst erstellten Symlink handelt, nicht um ein benutzererstelltes Skript.
Tech Stack
shell
Domain
build system
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Shell scriptingRPM packaging basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70