From nobody Wed May 22 11:13:15 2024 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4VkpZc5fNNz5LVSh; Wed, 22 May 2024 11:13:28 +0000 (UTC) (envelope-from oshogbo.vx@gmail.com) Received: from mail-oi1-f169.google.com (mail-oi1-f169.google.com [209.85.167.169]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "GTS CA 1D4" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4VkpZc3F7Nz52Sm; Wed, 22 May 2024 11:13:28 +0000 (UTC) (envelope-from oshogbo.vx@gmail.com) Authentication-Results: mx1.freebsd.org; none Received: by mail-oi1-f169.google.com with SMTP id 5614622812f47-3c9bcd57524so515449b6e.3; Wed, 22 May 2024 04:13:28 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1716376406; x=1716981206; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=cgjujC0UUJyh4d7FaanX51+b5g0HzJXF7FhfPoAGurU=; b=AyLioozYilUqcUwvyjJQIlVd8VOumGieoD+VmIeUkCAjm1zIsUiqk+z2+zQ/c8WOsM tTBDPd5MmMcsOpR7UYKP2D175K3/FHs0s8XNiRAIW/XfSVH4Akmd5ohemSN6ve/d6Qrf hKlzo8pS+PlDnWq/rMr1LyISopwax7iz6y9mTFnvvfRH7PZOP5sCjr0lwTOVwsZoIIMR 4CbooQqipZ+mnep4aE4Jtp3kwCevxiNIJSHjx7EgX0sWYqsgtlswTOmG9bFvKYMc2bPC E8QhEchI/Jc0KInsw0gmhsTqAOWSQ3YCE9BxKZ77/mYik5OHohIes+Px83dokZuuQ/pM THDw== X-Forwarded-Encrypted: i=1; AJvYcCVPHzy/0/Pkx3xy78/oLlmpWfnEslWgrMQnAWfq0VjQEMNSZY6Osw450ZPziU457MIS6OqIsvIEVkq2vmm9Zv+DNVdsqxLfdl/jmcfvCLQHJQjpHs4oOcPfxUgUg0HFenR44Bpjc/yPsM8SU3zMReQM36blMZLyipVsc9hlwZ5tSWeOzg8VWNxcvZFNmgM= X-Gm-Message-State: AOJu0Yx4DqMnsRj7H3YRjQRQHu5ltQk1wjpsJx7WFVVEnYEMuPrQiMNW WKkSX/3t36Yxyhjcw3V7WiYXXbbF23AH8wo8dM80THeoewSg4oFfdCWOA3LTQjw+dKfA+LC/2hr B4OwlWDF8vl/brO5vf+3J7XAba4GS7ypbDpfhng== X-Google-Smtp-Source: AGHT+IFFHzGAK4QkLstG/7W+PH1xrygUSLOaBvdC0sEFl1/jNxXchnxsQRUR7WbpgWNdxfxvOpAvUCD+DqsY93uFFtU= X-Received: by 2002:a05:6871:58a:b0:23b:3428:725a with SMTP id 586e51a60fabf-24c68b66638mr1998173fac.1.1716376406389; Wed, 22 May 2024 04:13:26 -0700 (PDT) List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: X-BeenThere: dev-commits-src-all@freebsd.org Sender: owner-dev-commits-src-all@FreeBSD.org MIME-Version: 1.0 References: <202405212004.44LK4cBG042804@gitrepo.freebsd.org> <86r0dugmik.fsf@peasant.tower.home> In-Reply-To: <86r0dugmik.fsf@peasant.tower.home> From: Mariusz Zaborski Date: Wed, 22 May 2024 13:13:15 +0200 Message-ID: Subject: Re: git: 408957613bc0 - main - Regen To: Dmitry Salychev Cc: Lexi Winter , src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Content-Type: multipart/alternative; boundary="00000000000000fe260619090601" X-Spamd-Bar: ---- X-Rspamd-Pre-Result: action=no action; module=replies; Message is reply to one we originated X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[]; TAGGED_FROM(0.00)[]; ASN(0.00)[asn:15169, ipnet:209.85.128.0/17, country:US] X-Rspamd-Queue-Id: 4VkpZc3F7Nz52Sm --00000000000000fe260619090601 Content-Type: text/plain; charset="UTF-8" Hello, Not really as a mistake :) The files in question have many such commits (not only committed by me). So I have assumed this is a common practice. I'm happy to provide more details next time. However, I don't plan to revert it at this point as it seems pointless to complicate the log even more. For freebsd32: e90259af59a1 regen 38bfddf597b1 regen 7893419d492c Remove never implemented sbrk and sstk syscalls 30cdbb5833f6 freebsd32: Remove mac_syscall from the unimpl list af93fea71038 timerfd: Move implementation from linux compat to sys/kern c7df87209634 Regen 9121945d7003 Regenerate sysent stuff after $FreeBSD$ removal 7b42f338d7a3 freebsd32: Regen 11cdffc60325 Regen 6a0a634590e4 Regen f99cc5a38947 sysent: regen after 52a1d90c8bfe, posix_fadvise in capmode 8601fca7895c sysent: regen for syscallarg_t 0910a41ef3b9 Revert "syscallarg_t: Add a type for system call arguments" 3889fb8af0b6 sysent: regen for syscallarg_t b7c55487ff2f Regen 5c1835b1d4b5 syscalls: regen 7fb006e7d61e syscalls: regen c2996f8ad930 syscalls: regen 30bc335a0501 freebsd32: regen 97fdcfcaf799 freebsd32: remove unnecessicary headers bcd07aaa4ebd freebsd32: regen 8460d3e9e6cb freebsd32: regen And for freebsd 408957613bc0 (HEAD -> main, origin/main, origin/HEAD) Regen 663838451ec6 syscalls: regen 38bfddf597b1 regen 7893419d492c Remove never implemented sbrk and sstk syscalls af93fea71038 timerfd: Move implementation from linux compat to sys/kern c7df87209634 Regen 9121945d7003 Regenerate sysent stuff after $FreeBSD$ removal 9abba78acc2a syscalls: regenerate 11cdffc60325 Regen 6a0a634590e4 Regen f99cc5a38947 sysent: regen after 52a1d90c8bfe, posix_fadvise in capmode 8601fca7895c sysent: regen for syscallarg_t 0910a41ef3b9 Revert "syscallarg_t: Add a type for system call arguments" 3889fb8af0b6 sysent: regen for syscallarg_t b7c55487ff2f Regen 5c1835b1d4b5 syscalls: regen 7fb006e7d61e syscalls: regen c2996f8ad930 syscalls: regen e58e9a8cbd7c syscalls: regen 439c3d95638a Regen da9fe3529b3d Regen after 0dc332bff200 273728b1257f Regen 2c9764f36b6f regen syscall files after d51198d63b63 d89c1c461ce3 Reserve gaps in syscall numbers for local use 65a524b499ae Remove documentation of unimplemented syscalls 20321e622515 Regenerate syscall files after reallocation of aio_writev/aio_readv 1868a91fac5e Regenerate syscall files after addition of aio_writev/aio_readv 9dd48b87e6ca Regen. 0c5bd5f99320 Regen after r366145. Thanks, Mariusz On Wed, 22 May 2024 at 12:37, Dmitry Salychev wrote: > > Lexi Winter writes: > > > [[PGP Signed Part:Undecided]] > > Mariusz Zaborski: > >> The branch main has been updated by oshogbo: > >> > >> URL: > https://cgit.FreeBSD.org/src/commit/?id=408957613bc065c4444812c3ae276ace03697ebd > >> > >> commit 408957613bc065c4444812c3ae276ace03697ebd > >> Author: Mariusz Zaborski > >> AuthorDate: 2024-05-21 20:02:50 +0000 > >> Commit: Mariusz Zaborski > >> CommitDate: 2024-05-21 20:03:20 +0000 > >> > >> Regen > >> --- > >> sys/compat/freebsd32/freebsd32_sysent.c | 2 +- > >> sys/kern/init_sysent.c | 2 +- > >> 2 files changed, 2 insertions(+), 2 deletions(-) > > > > i'm sorry if this is considered accepted practice in FreeBSD (i'm not a > > committer) but could i please request more useful commit messages than > > "Regen" for things like this? > > > > when looking through 'git log', especially 'git log --format=oneline', > > it's really not clear what this commit actually does, meaning if i'm > > looking through the log for commits related to a problem i'm having, i > > have to examine commits like this by hand to see if they're related. > > > > it would save me (and other people, i suppose) a lot of time if the > > commit message could at least mention what it's touching, e.g. "Regen > > sys/compat/freebsd32/freebsd32_sysent.c". > > > > [[End of PGP Signed Part]] > > Looks like it slipped in by mistake. I expected it to be reverted and > re-worded. > > Regards, > Dmitry > > -- > https://wiki.freebsd.org/DmitrySalychev > --00000000000000fe260619090601 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Hello,

Not really as a mistake :) The files in ques= tion have many such commits (not only committed by me). So I have assumed t= his is a common practice.
I'm happy to provide more details next tim= e. However, I don't plan to revert it at this point as it seems pointle= ss to complicate the log even more.

For freebsd32:
e90259af59a1 r= egen
38bfddf597b1 regen
7893419d492c Remove never implemented sbrk an= d sstk syscalls
30cdbb5833f6 freebsd32: Remove mac_syscall from the unim= pl list
af93fea71038 timerfd: Move implementation from linux compat to s= ys/kern
c7df87209634 Regen
9121945d7003 Regenerate sysent stuff after= $FreeBSD$ removal
7b42f338d7a3 freebsd32: Regen
11cdffc60325 Regen6a0a634590e4 Regen
f99cc5a38947 sysent: regen after 52a1d90c8bfe, posi= x_fadvise in capmode
8601fca7895c sysent: regen for syscallarg_t
0910= a41ef3b9 Revert "syscallarg_t: Add a type for system call arguments&qu= ot;
3889fb8af0b6 sysent: regen for syscallarg_t
b7c55487ff2f Regen5c1835b1d4b5 syscalls: regen
7fb006e7d61e syscalls: regen
c2996f8ad9= 30 syscalls: regen
30bc335a0501 freebsd32: regen
97fdcfcaf799 freebsd= 32: remove unnecessicary headers
bcd07aaa4ebd freebsd32: regen
8460d3= e9e6cb freebsd32: regen

And for freebsd
408957613bc0 (HEAD -> = main, origin/main, origin/HEAD) Regen
663838451ec6 syscalls: regen
38= bfddf597b1 regen
7893419d492c Remove never implemented sbrk and sstk sys= calls
af93fea71038 timerfd: Move implementation from linux compat to sys= /kern
c7df87209634 Regen
9121945d7003 Regenerate sysent stuff after $= FreeBSD$ removal
9abba78acc2a syscalls: regenerate
11cdffc60325 Regen=
6a0a634590e4 Regen
f99cc5a38947 sysent: regen after 52a1d90c8bfe, po= six_fadvise in capmode
8601fca7895c sysent: regen for syscallarg_t
09= 10a41ef3b9 Revert "syscallarg_t: Add a type for system call arguments&= quot;
3889fb8af0b6 sysent: regen for syscallarg_t
b7c55487ff2f Regen<= br>5c1835b1d4b5 syscalls: regen
7fb006e7d61e syscalls: regen
c2996f8a= d930 syscalls: regen
e58e9a8cbd7c syscalls: regen
439c3d95638a Regen<= br>da9fe3529b3d Regen after 0dc332bff200
273728b1257f Regen
2c9764f36= b6f regen syscall files after d51198d63b63
d89c1c461ce3 Reserve gaps in = syscall numbers for local use
65a524b499ae Remove documentation of unimp= lemented syscalls
20321e622515 Regenerate syscall files after reallocati= on of aio_writev/aio_readv
1868a91fac5e Regenerate syscall files after a= ddition of aio_writev/aio_readv
9dd48b87e6ca Regen.
0c5bd5f99320 Rege= n after r366145.

Thanks,
Mariusz

On Wed, 22 May 2024 at 12:37, Dmi= try Salychev <dsl@freebsd.org>= wrote:

Lexi Winter <lexi@l= e-fay.org> writes:

> [[PGP Signed Part:Undecided]]
> Mariusz Zaborski:
>> The branch main has been updated by oshogbo:
>>
>> URL: http= s://cgit.FreeBSD.org/src/commit/?id=3D408957613bc065c4444812c3ae276ace03697= ebd
>>
>> commit 408957613bc065c4444812c3ae276ace03697ebd
>> Author:=C2=A0 =C2=A0 =C2=A0Mariusz Zaborski <oshogbo@FreeBSD.or= g>
>> AuthorDate: 2024-05-21 20:02:50 +0000
>> Commit:=C2=A0 =C2=A0 =C2=A0Mariusz Zaborski <oshogbo@FreeBSD.or= g>
>> CommitDate: 2024-05-21 20:03:20 +0000
>>
>>=C2=A0 =C2=A0 =C2=A0Regen
>> ---
>>=C2=A0 sys/compat/freebsd32/freebsd32_sysent.c | 2 +-
>>=C2=A0 sys/kern/init_sysent.c=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 | 2 +-
>>=C2=A0 2 files changed, 2 insertions(+), 2 deletions(-)
>
> i'm sorry if this is considered accepted practice in FreeBSD (i= 9;m not a
> committer) but could i please request more useful commit messages than=
> "Regen" for things like this?
>
> when looking through 'git log', especially 'git log --form= at=3Doneline',
> it's really not clear what this commit actually does, meaning if i= 'm
> looking through the log for commits related to a problem i'm havin= g, i
> have to examine commits like this by hand to see if they're relate= d.
>
> it would save me (and other people, i suppose) a lot of time if the > commit message could at least mention what it's touching, e.g. &qu= ot;Regen
> sys/compat/freebsd32/freebsd32_sysent.c".
>
> [[End of PGP Signed Part]]

Looks like it slipped in by mistake. I expected it to be reverted and
re-worded.

Regards,
Dmitry

--
https://wiki.freebsd.org/DmitrySalychev
--00000000000000fe260619090601--