[PR #88] [MERGED] Fix path to /usr/bin/rm #226

New Issue

claunia · 2026-01-29T16:31:22Z

claunia commented

2026-01-29 16:31:22 +00:00

📋 Pull Request Information

Original PR: https://github.com/quamotion/dotnet-packaging/pull/88
Author: @qmfrederik
Created: 12/16/2018
Status: ✅ Merged
Merged: 12/17/2018
Merged by: @qmfrederik

Base: master ← Head: fixes/rm

📝 Commits (2)

5af9edc Uninstall the package as part of the tests, too
adc4092 Use /bin/rm instead of /usr/bin/rm

📊 Changes

3 files changed (+18 additions, -3 deletions)

View changed files

📝 Packaging.Targets.Tests/Deb/DebPackageCreatorTests.cs (+2 -2)
📝 Packaging.Targets/Deb/DebPackageCreator.cs (+1 -1)
📝 demo/cliscd/run-tests.sh (+15 -0)

📄 Description

At least on Ubuntu 18.04, /usr/bin/rm doesn't exist - but /bin/rm does. The /usr/bin copy comes from the safe-rm package. We could just declare a dependency on this package, but this is in universe and this repo is not enabled by default.

Fixes #55

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/quamotion/dotnet-packaging/pull/88 **Author:** [@qmfrederik](https://github.com/qmfrederik) **Created:** 12/16/2018 **Status:** ✅ Merged **Merged:** 12/17/2018 **Merged by:** [@qmfrederik](https://github.com/qmfrederik) **Base:** `master` ← **Head:** `fixes/rm` --- ### 📝 Commits (2) - [`5af9edc`](https://github.com/quamotion/dotnet-packaging/commit/5af9edc3df97658142be891e68b5aaa0e5e7d949) Uninstall the package as part of the tests, too - [`adc4092`](https://github.com/quamotion/dotnet-packaging/commit/adc4092a503613e1000b906d92a5d7fe7529e881) Use /bin/rm instead of /usr/bin/rm ### 📊 Changes **3 files changed** (+18 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `Packaging.Targets.Tests/Deb/DebPackageCreatorTests.cs` (+2 -2) 📝 `Packaging.Targets/Deb/DebPackageCreator.cs` (+1 -1) 📝 `demo/cliscd/run-tests.sh` (+15 -0) </details> ### 📄 Description At least on Ubuntu 18.04, /usr/bin/rm doesn't exist - but /bin/rm does. The /usr/bin copy comes from the safe-rm package. We could just declare a dependency on this package, but this is in universe and this repo is not enabled by default. Fixes #55 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

claunia added the pull-request label 2026-01-29 16:31:22 +00:00

claunia closed this issue

2026-01-29 16:31:23 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/dotnet-packaging#226