| Message ID | 207156d802739bf6225591450dfc19b710be0350.1710857220.git.anand.jain@oracle.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | btrfs-progs: snapshot fix user message | expand |
On Tue, Mar 19, 2024 at 07:41:29PM +0530, Anand Jain wrote: > The fstests depend on the output message of the create snapshot command, > and if it's changed, the tests fail. Bring back the original messages, > as they are also grammatically correct. I removed the 'a' for consistency with other messages. That fstests depend too much on what applications return is not a new thing and the breakage will happen once the user-facing error messages get updated as they get the priority.
On 3/26/24 03:06, David Sterba wrote: > On Tue, Mar 19, 2024 at 07:41:29PM +0530, Anand Jain wrote: >> The fstests depend on the output message of the create snapshot command, >> and if it's changed, the tests fail. Bring back the original messages, >> as they are also grammatically correct. > > I removed the 'a' for consistency with other messages. That fstests > depend too much on what applications return is not a new thing and the > breakage will happen once the user-facing error messages get updated as > they get the priority. Alright, I wasn't aware it was an intentional change. We need to update the fstests test cases. I'll take care of it. Thanks, Anand
diff --git a/cmds/subvolume.c b/cmds/subvolume.c index 11bb5f560ad5..6516ea981d4d 100644 --- a/cmds/subvolume.c +++ b/cmds/subvolume.c @@ -779,11 +779,11 @@ static int cmd_subvolume_snapshot(const struct cmd_struct *cmd, int argc, char * if (readonly) pr_verbose(LOG_DEFAULT, - "Create readonly snapshot of '%s' in '%s/%s'\n", + "Create a readonly snapshot of '%s' in '%s/%s'\n", subvol, dstdir, newname); else pr_verbose(LOG_DEFAULT, - "Create snapshot of '%s' in '%s/%s'\n", + "Create a snapshot of '%s' in '%s/%s'\n", subvol, dstdir, newname); out:
The fstests depend on the output message of the create snapshot command, and if it's changed, the tests fail. Bring back the original messages, as they are also grammatically correct. Fixes: 5f87b467a9e7 ("btrfs-progs: subvolume: output the prompt line only when the ioctl succeeded") Signed-off-by: Anand Jain <anand.jain@oracle.com> --- cmds/subvolume.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)