[pacman-dev] Fix srcinfo sometimes printing double newline at the end
Allan McRae
allan at archlinux.org
Thu Jun 25 23:43:39 UTC 2020
On 26/6/20 9:27 am, Denton Liu wrote:
> Hi Marti,
>
> On Fri, Jun 26, 2020 at 01:09:54AM +0200, Martin Rys wrote:
>> Hi,
>> this is a bit of an OCD thing, but `makepkg --printsrcinfo` usually
>> generates a file with two newlines at the end.
>
> I agree with you, this bugs me a lot too.
>
> [...]
>
>> Which seems to always print out a newline, so I believe the
>> srcinfo_close_section function is not needed there and simply
>> removing it and the two lines that call it would solve this issue, as
>> per the patchfile in the attachment.
>
> I proposed a similar patch recently[0] but it was rejected as the
> newline separators are apparently part of the spec so they should be
> retained[1]. I sent out another patch after that only removes the final
> newline but it wasn't reviewed so I thought that the pacman devs thought
> it was a non-issue. Since you seem to agree with me, though, I'll resend
> that patch and hopefully it'll be reviewed this time around.
>
Please do not resend patches. We have a patch tracking system, which
means patches are never lost.
I have given this patch a very low priority. That means with limited
time available to me, I will likely review other patches first. But it
will get considered at some stage.
Allan
More information about the pacman-dev
mailing list