Summary: | devel/git: Command 'git maintenance start' fails with 'file name must be specified for replace' | ||
---|---|---|---|
Product: | Ports & Packages | Reporter: | Mark Waite <mark.earl.waite> |
Component: | Individual Port(s) | Assignee: | Renato Botelho <garga> |
Status: | Closed FIXED | ||
Severity: | Affects Many People | CC: | emaste, lwhsu |
Priority: | --- | ||
Version: | Latest | ||
Hardware: | Any | ||
OS: | Any |
Description
Mark Waite
2021-12-28 04:59:40 UTC
A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=82c314474fce8cd4a9ea5cc95d0ca4f6165e6acc commit 82c314474fce8cd4a9ea5cc95d0ca4f6165e6acc Author: Renato Botelho <garga@FreeBSD.org> AuthorDate: 2022-08-29 10:15:42 +0000 Commit: Renato Botelho <garga@FreeBSD.org> CommitDate: 2022-08-29 10:49:23 +0000 devel/git: Fix 'git maintenance start' Add a patch from upstream to fix 'git maintenance start'. Here follow upstream commit message: While cron is specified by POSIX, there are a wide variety of implementations in use. On FreeBSD, the cron implementation requires a file name argument: if the user wants to edit standard input, they must specify "-". However, this notation is not specified by POSIX, allowing the possibility that making such a change may break other, less common implementations. Since POSIX tells us that cron must accept a file name argument, let's solve this problem by specifying a temporary file instead. This will ensure that we work with the vast majority of implementations. Note that because delete_tempfile closes the file for us, we should not call fclose here on the handle, since doing so will introduce a double free. PR: 260746 Reported by: Mark Waite <mark.earl.waite@gmail.com> Obtained from: https://lore.kernel.org/git/1dd29f43-1a8e-eb69-3320-7f5140a0e18e@github.com/T/#me47087c1f9034c82da8eae83db18cc6114c2c1e5 Sponsored by: Rubicon Communications, LLC ("Netgate") devel/git/Makefile | 1 + devel/git/files/patch-fix-maintenance.diff (new) | 112 +++++++++++++++++++++++ 2 files changed, 113 insertions(+) Patch from upstream is now applied on ports tree. Package version 2.37.2_1 is fixed. Thanks! |