Hello community,
here is the log from the commit of package xfsprogs for openSUSE:Leap:15.2
checked in at 2020-04-14 14:18:49
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Leap:15.2/xfsprogs (Old)
and /work/SRC/openSUSE:Leap:15.2/.xfsprogs.new.3248 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "xfsprogs"
Tue Apr 14 14:18:49 2020 rev:36 rq:792905 version:4.15.0
Changes:
--------
--- /work/SRC/openSUSE:Leap:15.2/xfsprogs/xfsprogs.changes 2020-02-27
06:41:32.565614533 +0100
+++ /work/SRC/openSUSE:Leap:15.2/.xfsprogs.new.3248/xfsprogs.changes
2020-04-14 14:18:51.157179170 +0200
@@ -1,0 +2,21 @@
+Thu Mar 26 19:28:00 UTC 2020 - Anthony Iliopoulos <ailiopou...@suse.com>
+
+- xfs_quota: reformat commands in the manpage so that fstest can check
+ that each command is actually documented (bsc#1167206)
+ * Add xfsprogs-man-reformat-xfs_quota-commands-in-the-manpage-for-t.patch
+
+-------------------------------------------------------------------
+Thu Mar 26 19:06:20 UTC 2020 - Anthony Iliopoulos <ailiopou...@suse.com>
+
+- xfs_db: document some missing commands ('attr_set', 'attr_remove',
+ 'logformat') in the manpage (bsc#1167205)
+ * Add xfsprogs-man-document-some-missing-xfs_db-commands.patch
+
+-------------------------------------------------------------------
+Thu Mar 26 18:12:14 UTC 2020 - Anthony Iliopoulos <ailiopou...@suse.com>
+
+- xfs_io: allow size to be suffixed by k,m,g for kilobytes, megabytes or
+ gigabytes respectively for the copy_range command (bsc#1158630)
+ * Add xfsprogs-xfs_io-Make-copy_range-arguments-understand-iB-values.patch
+
+-------------------------------------------------------------------
New:
----
xfsprogs-man-document-some-missing-xfs_db-commands.patch
xfsprogs-man-reformat-xfs_quota-commands-in-the-manpage-for-t.patch
xfsprogs-xfs_io-Make-copy_range-arguments-understand-iB-values.patch
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ xfsprogs.spec ++++++
--- /var/tmp/diff_new_pack.zFgIl2/_old 2020-04-14 14:18:52.553180212 +0200
+++ /var/tmp/diff_new_pack.zFgIl2/_new 2020-04-14 14:18:52.565180221 +0200
@@ -57,6 +57,9 @@
Patch24:
xfsprogs-xfs-move-inode-extent-size-hint-validation-to-libxfs.patch
Patch25:
xfsprogs-xfs_repair-use-libxfs-extsize-cowextsize-validation-.patch
Patch26: xfsprogs-mkfs-validate-extent-size-hint-parameters.patch
+Patch27:
xfsprogs-xfs_io-Make-copy_range-arguments-understand-iB-values.patch
+Patch28: xfsprogs-man-document-some-missing-xfs_db-commands.patch
+Patch29:
xfsprogs-man-reformat-xfs_quota-commands-in-the-manpage-for-t.patch
BuildRequires: autoconf
BuildRequires: automake
@@ -132,6 +135,9 @@
%patch24 -p1
%patch25 -p1
%patch26 -p1
+%patch27 -p1
+%patch28 -p1
+%patch29 -p1
%endif
%build
++++++ xfsprogs-man-document-some-missing-xfs_db-commands.patch ++++++
>From 946352013f75590d16d6a93b06a805330106b4c5 Mon Sep 17 00:00:00 2001
From: "Darrick J. Wong" <darrick.w...@oracle.com>
Date: Thu, 30 Jan 2020 13:39:33 -0500
Subject: [PATCH] man: document some missing xfs_db commands
Git-commit: 946352013f75590d16d6a93b06a805330106b4c5
Patch-mainline: v5.5.0-rc1
References: bsc#1167205
The 'attr_set', 'attr_remove', and 'logformat' commands in xfs_db were
not documented. Add sections about them to the manpage.
Signed-off-by: Darrick J. Wong <darrick.w...@oracle.com>
Reviewed-by: Christoph Hellwig <h...@lst.de>
Signed-off-by: Eric Sandeen <sand...@sandeen.net>
Acked-by: Anthony Iliopoulos <ailiopou...@suse.com>
---
db/attrset.c | 4 ++--
man/man8/xfs_db.8 | 58 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 60 insertions(+), 2 deletions(-)
diff --git a/db/attrset.c b/db/attrset.c
index ad3c8f37..bbd79cb8 100644
--- a/db/attrset.c
+++ b/db/attrset.c
@@ -35,11 +35,11 @@ static void attrset_help(void);
static const cmdinfo_t attr_set_cmd =
{ "attr_set", "aset", attr_set_f, 1, -1, 0,
- N_("[-r|-s|-p|-u] [-n] [-R|-C] [-v n] name"),
+ N_("[-r|-s|-u] [-n] [-R|-C] [-v n] name"),
N_("set the named attribute on the current inode"), attrset_help };
static const cmdinfo_t attr_remove_cmd =
{ "attr_remove", "aremove", attr_remove_f, 1, -1, 0,
- N_("[-r|-s|-p|-u] [-n] name"),
+ N_("[-r|-s|-u] [-n] name"),
N_("remove the named attribute from the current inode"), attrset_help
};
static void
diff --git a/man/man8/xfs_db.8 b/man/man8/xfs_db.8
index 37018a7a..673ec7e4 100644
--- a/man/man8/xfs_db.8
+++ b/man/man8/xfs_db.8
@@ -179,6 +179,57 @@ Set current address to the AGI block for allocation group
.IR agno .
If no argument is given, use the current allocation group.
.TP
+.BI "attr_remove [\-r|\-u|\-s] [\-n] " name
+Remove the specified extended attribute from the current file.
+.RS 1.0i
+.TP 0.4i
+.B \-r
+Sets the attribute in the root namespace.
+Only one namespace option can be specified.
+.TP
+.B \-u
+Sets the attribute in the user namespace.
+Only one namespace option can be specified.
+.TP
+.B \-s
+Sets the attribute in the secure namespace.
+Only one namespace option can be specified.
+.TP
+.B \-n
+Do not enable 'noattr2' mode on V4 filesystems.
+.RE
+.TP
+.BI "attr_set [\-r|\-u|\-s] [\-n] [\-R|\-C] [\-v " namelen "] " name
+Sets an extended attribute on the current file with the given name.
+.RS 1.0i
+.TP 0.4i
+.B \-r
+Sets the attribute in the root namespace.
+Only one namespace option can be specified.
+.TP
+.B \-u
+Sets the attribute in the user namespace.
+Only one namespace option can be specified.
+.TP
+.B \-s
+Sets the attribute in the secure namespace.
+Only one namespace option can be specified.
+.TP
+.B \-n
+Do not enable 'noattr2' mode on V4 filesystems.
+.TP
+.B \-R
+Replace the attribute.
+The command will fail if the attribute does not already exist.
+.TP
+.B \-C
+Create the attribute.
+The command will fail if the attribute already exists.
+.TP
+.B \-v
+Set the attribute value to a string of this length containing the letter 'v'.
+.RE
+.TP
.B b
See the
.B back
@@ -690,6 +741,13 @@ Start logging output to
.IR filename ,
stop logging, or print the current logging status.
.TP
+.BI "logformat [\-c " cycle "] [\-s " sunit "]"
+Reformats the log to the specified log cycle and log stripe unit.
+This has the effect of clearing the log destructively.
+If the log cycle is not specified, the log is reformatted to the current cycle.
+If the log stripe unit is not specified, the stripe unit from the filesystem
+superblock is used.
+.TP
.BI "metadump [\-egow] " filename
Dumps metadata to a file. See
.BR xfs_metadump (8)
--
2.16.4
++++++ xfsprogs-man-reformat-xfs_quota-commands-in-the-manpage-for-t.patch
++++++
>From af0876e74c7f2764377b70e29023a0bdf291bb42 Mon Sep 17 00:00:00 2001
From: "Darrick J. Wong" <darrick.w...@oracle.com>
Date: Thu, 30 Jan 2020 13:39:18 -0500
Subject: [PATCH] man: reformat xfs_quota commands in the manpage for testing
Git-commit: af0876e74c7f2764377b70e29023a0bdf291bb42
Patch-mainline: v5.5.0-rc1
References: bsc#1167206
Reformat the xfs_quota commands listed in the xfs_quota.8 manpage so
that we can implement a fstest that checks that each command actually
has a section in the manpage.
Signed-off-by: Darrick J. Wong <darrick.w...@oracle.com>
Reviewed-by: Christoph Hellwig <h...@lst.de>
Signed-off-by: Eric Sandeen <sand...@sandeen.net>
Acked-by: Anthony Iliopoulos <ailiopou...@suse.com>
---
man/man8/xfs_quota.8 | 15 +++++----------
1 file changed, 5 insertions(+), 10 deletions(-)
diff --git a/man/man8/xfs_quota.8 b/man/man8/xfs_quota.8
index d9c5f649..e6fe7cd1 100644
--- a/man/man8/xfs_quota.8
+++ b/man/man8/xfs_quota.8
@@ -228,8 +228,7 @@ option sends the output to
.I file
instead of stdout.
.HP
-.B
-free
+.B free
[
.B \-bir
] [
@@ -398,8 +397,7 @@ option reports information without the header line. The
.B \-t
option performs a terse report.
.HP
-.B
-state
+.B state
[
.B \-gpu
] [
@@ -420,8 +418,7 @@ instead of stdout. The
.B \-a
option reports state on all filesystems and not just the current path.
.HP
-.B
-limit
+.B limit
[
.BR \-g " | " \-p " | " \-u
]
@@ -519,8 +516,7 @@ identified by the current path.
Quota must not be enabled on the filesystem, else this operation will
report an error.
.HP
-.B
-dump
+.B dump
[
.BR \-g " | " \-p " | " \-u
] [
@@ -547,8 +543,7 @@ The file must be in the format produced by the
.B dump
command.
.HP
-.B
-quot
+.B quot
[
.BR \-g " | " \-p " | " \-u
] [
--
2.16.4
++++++ xfsprogs-xfs_io-Make-copy_range-arguments-understand-iB-values.patch
++++++
>From 25b4549c8b54134106cff094aa098eab9e86eee7 Mon Sep 17 00:00:00 2001
From: Goldwyn Rodrigues <rgold...@suse.com>
Date: Thu, 31 May 2018 14:09:55 -0500
Subject: [PATCH] xfs_io: Make copy_range arguments understand *iB values
Git-commit: 25b4549c8b54134106cff094aa098eab9e86eee7
Patch-mainline: v4.17.0-rc1
References: bsc#1158630
Arguments such as 2MiB or 2M are converted to 2 because copy_range uses
strtoull(). Convert strtoull() to cvtnum().
Signed-off-by: Goldwyn Rodrigues <rgold...@suse.com>
Reviewed-by: Eric Sandeen <sand...@redhat.com>
Signed-off-by: Eric Sandeen <sand...@sandeen.net>
Acked-by: Anthony Iliopoulos <ailiopou...@suse.com>
---
io/copy_file_range.c | 28 +++++++++++++++-------------
1 file changed, 15 insertions(+), 13 deletions(-)
diff --git a/io/copy_file_range.c b/io/copy_file_range.c
index 99fba20a..ebd1ec76 100644
--- a/io/copy_file_range.c
+++ b/io/copy_file_range.c
@@ -47,7 +47,7 @@ copy_range_help(void)
* glibc buffered copy fallback.
*/
static loff_t
-copy_file_range_cmd(int fd, loff_t *src, loff_t *dst, size_t len)
+copy_file_range_cmd(int fd, long long *src, long long *dst, long long len)
{
loff_t ret;
@@ -89,34 +89,36 @@ copy_dst_truncate(void)
static int
copy_range_f(int argc, char **argv)
{
- loff_t src = 0;
- loff_t dst = 0;
+ long long src = 0;
+ long long dst = 0;
size_t len = 0;
- char *sp;
int opt;
int ret;
int fd;
+ size_t fsblocksize, fssectsize;
+
+ init_cvtnum(&fsblocksize, &fssectsize);
while ((opt = getopt(argc, argv, "s:d:l:")) != -1) {
switch (opt) {
case 's':
- src = strtoull(optarg, &sp, 10);
- if (!sp || sp == optarg) {
- printf(_("invalid source offset -- %s\n"), sp);
+ src = cvtnum(fsblocksize, fssectsize, optarg);
+ if (src < 0) {
+ printf(_("invalid source offset -- %s\n"),
optarg);
return 0;
}
break;
case 'd':
- dst = strtoull(optarg, &sp, 10);
- if (!sp || sp == optarg) {
- printf(_("invalid destination offset -- %s\n"),
sp);
+ dst = cvtnum(fsblocksize, fssectsize, optarg);
+ if (dst < 0) {
+ printf(_("invalid destination offset -- %s\n"),
optarg);
return 0;
}
break;
case 'l':
- len = strtoull(optarg, &sp, 10);
- if (!sp || sp == optarg) {
- printf(_("invalid length -- %s\n"), sp);
+ len = cvtnum(fsblocksize, fssectsize, optarg);
+ if (len < 0) {
+ printf(_("invalid length -- %s\n"), optarg);
return 0;
}
break;
--
2.16.4
