Integration of VSCodium installation script
Due to change in ownership/assignee, this MR shall replace an older MR that got stalled.
Merge request reports
Activity
requested review from @daissi
assigned to @sietze.vanbuuren
@daissi As discussed in the old MR, I created a new branch and a corresponding MR.
I assume that it doesn't make any sense to try to merge this into
v2024dev3
, so I switched tov2025dev2
. Please, allow me fix the conflicts and include your suggestions from the older MR. Then we can continue the review here.Edited by Sietze van Buurenadded 26 commits
-
4a874b71...15796547 - 24 commits from branch
apertis/v2025dev2
- 0ab8ba96 - Integration of VSCodium installation script
- e57d9767 - Release apertis-dev-tools version 0.2024dev3
-
4a874b71...15796547 - 24 commits from branch
@daissi I resolved the conflicts and included your suggestions. Please let me know, if there are any open points.
I think we can close the old MR now.
Edited by Sietze van Buuren- Resolved by Sietze van Buuren
- Resolved by Sietze van Buuren
The file
debian/postinst
should be renamed todebian/vscodium-ide-setup.postinst
.By default files like
debian/postinst
anddebian/install
apply to the first binary package listed indebian/control
. Since here we want this script to be applied only when installingvscodium-ide-setup
then we should specify in its name to which package it applies.
added 1 commit
- 07e45ae6 - Adjust postint filename for vscodium-ide-setup
mentioned in merge request !62 (closed)
I proposed some changes in !71 (merged)
mentioned in merge request !72 (merged)
added 1 commit
- 35a266a1 - Simplify the process to download the apt key for vscodium
- Resolved by Dylan Aïssi