mirror of
https://github.com/netblue30/firejail.git
synced 2026-05-15 14:16:14 -06:00
[PR #6290] [MERGED] build: sort.py: add and require -i to edit in-place #5934
Labels
No labels
LTS merge
LTS merge
bug
bug
converted-to-discussion
doc-todo
documentation
duplicate
enhancement
file-transfer
firecfg
firejail-in-firejail
firetools
graphics
help wanted
information_old
installation
invalid
modif
moved
needinfo
networking
notabug
notourbug
old-version
overlayfs
packaging
profile-request
pull-request
question
question_old
removal
runtime-permissions
sandbox-ipc
security
stale
wiki
wiki
wontfix
wordpress
workaround
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/firejail#5934
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/netblue30/firejail/pull/6290
Author: @kmk3
Created: 3/24/2024
Status: ✅ Merged
Merged: 3/25/2024
Merged by: @kmk3
Base:
master← Head:build-sort-py-i📝 Commits (1)
f7c0502build: sort.py: add and require -i to edit in-place📊 Changes
1 file changed (+22 additions, -14 deletions)
View changed files
📝
contrib/sort.py(+22 -14)📄 Description
Similarly to
sed -iandperl -i.This allows checking if sort.py correctly sorts the relevant lines in a
profile without having to overwrite it, which makes debugging and
testing easier (for example, in #6261).
Note: If it finds items that are not sorted, it still sorts them, prints
the diff and returns an error.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.