From: Dmitry Smirnov <onlyjob-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org>
To: linux-nilfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: few patches from Debian
Date: Fri, 11 Apr 2014 17:49:43 +1000 [thread overview]
Message-ID: <2256155.lyPQvkPv8r@debstor> (raw)
[-- Attachment #1.1: Type: text/plain, Size: 4070 bytes --]
=2D-nextPart2933964.7gAWMFQZoV
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"
Dear NILFS team,
Thanks for your hard work and for new release of nilfs-tools.
I'd like to share two patches (attached) to fix some minor problems that I've
noticed:
* man.patch (with minor man pages corrections)
* fhs.patch (to install nilfs-* executables to "/usr/sbin" instead of "/sbin"
due to library dependencies in "/usr").
Please review and apply (if appropriate).
Also please CC to me as I'm not subscribed.
Thank you.
=2D-
All the best,
Dmitry Smirnov
GPG key : 4096R/53968D1B
=2D-nextPart2933964.7gAWMFQZoV
Content-Disposition: attachment; filename="fhs.patch"
Content-Transfer-Encoding: 7Bit
Content-Type: text/x-patch; charset="UTF-8"; name="fhs.patch"
Last-Update: 2014-04-11
Forwarded: yes
Author: Dmitry Smirnov <onlyjob-IGUgQLVVQiRCV4ILt04nZQ@public.gmane.org>
Description: install nilfs-* executables to /usr/sbin:
Fixes the following problems (thanks, `adequate`):
bin-or-sbin-binary-requires-usr-lib-library
/sbin/nilfs-clean => /usr/lib/x86_64-linux-gnu/libnilfscleaner.so.0
/sbin/nilfs-clean => /usr/lib/x86_64-linux-gnu/libnilfs.so.0
/sbin/nilfs-resize => /usr/lib/x86_64-linux-gnu/libnilfs.so.0
/sbin/nilfs-resize => /usr/lib/x86_64-linux-gnu/libnilfsgc.so.0
/sbin/nilfs-tune => /usr/lib/x86_64-linux-gnu/libnilfs.so.0
/sbin/nilfs_cleanerd => /usr/lib/x86_64-linux-gnu/libnilfs.so.0
/sbin/nilfs_cleanerd => /usr/lib/x86_64-linux-gnu/libnilfsgc.so.0
=2D-- a/configure.ac
+++ b/configure.ac
@@ -165,9 +165,10 @@
AC_SYS_LARGEFILE
# Install directories
AC_PREFIX_DEFAULT([/usr])
=2DAC_SUBST([sbindir], [/sbin])
+AC_SUBST([root_sbindir], [/sbin])
+AC_SUBST([sbindir], [${prefix}/sbin])
AC_SUBST([sysconfdir], [/etc])
AC_SUBST([localstatedir], [/var])
AC_CONFIG_FILES([Makefile
=2D-- a/sbin/mkfs/Makefile.am
+++ b/sbin/mkfs/Makefile.am
@@ -5,9 +5,9 @@
LDADD = -luuid $(LIB_BLKID) $(top_builddir)/lib/libnilfsfeature.la \
$(top_builddir)/lib/libmountchk.la \
$(top_builddir)/lib/libcrc32.la
=2Dsbin_PROGRAMS = mkfs.nilfs2
+root_sbin_PROGRAMS = mkfs.nilfs2
mkfs_nilfs2_SOURCES = mkfs.c bitops.c mkfs.h
EXTRA_DIST = .gitignore
=2D-- a/sbin/mount/Makefile.am
+++ b/sbin/mount/Makefile.am
@@ -10,9 +10,9 @@
AM_CPPFLAGS = -I$(top_srcdir)/include
LDADD = $(top_builddir)/lib/librealpath.la \
$(top_builddir)/lib/libcleanerexec.la $(LIB_MOUNT) $(LIB_SELINUX)
=2Dsbin_PROGRAMS = mount.nilfs2 umount.nilfs2
+root_sbin_PROGRAMS = mount.nilfs2 umount.nilfs2
if CONFIG_LIBMOUNT
COMMONSOURCES += mount_attrs.c
=2D-nextPart2933964.7gAWMFQZoV
Content-Disposition: attachment; filename="man.patch"
Content-Transfer-Encoding: 7Bit
Content-Type: text/x-patch; charset="UTF-8"; name="man.patch"
Last-Update: 2014-04-11
Forwarded: yes
Author: Dmitry Smirnov <onlyjob-IGUgQLVVQiRCV4ILt04nZQ@public.gmane.org>
Description: minor man pages lintianisation
spelling-error-in-manpage
hyphen-used-as-minus-sign
=2D-- a/man/nilfs-clean.8
+++ b/man/nilfs-clean.8
@@ -19,9 +19,9 @@
sent to and performed on the \fBnilfs_cleanerd\fP(8) program. If no
commands are specified, \fBnilfs-clean\fP just triggers a one-pass
cleaning.
.PP
=2DWhen \fIdevice\fP is ommitted, \fBnilfs-clean\fP selects an active
+When \fIdevice\fP is omitted, \fBnilfs-clean\fP selects an active
NILFS2 file system in the system.
.PP
This command is valid only for mounted NILFS2 file systems, and
will fail if the \fIdevice\fP has no active mounts.
=2D-- a/man/umount.nilfs2.8
+++ b/man/umount.nilfs2.8
@@ -41,9 +41,9 @@
Verbose mode.
.SH THE LOOP DEVICE
.B umount.nilfs2
will free the loop device (if any) associated with the mount, in case
=2Dit finds the option `loop=...' in \fI/etc/mtab\fP. The -d option,
+it finds the option `loop=...' in \fI/etc/mtab\fP. The \-d option,
which also allows \fBumount\fP(8) to free the loop device without the
`loop=...' option, is not supported.
.SH AUTHOR
.B umount.nilfs2
=2D-nextPart2933964.7gAWMFQZoV--
This is a multi-part message in MIME format.
[-- Attachment #1.2: Type: text/plain, Size: 500 bytes --]
Dear NILFS team,
Thanks for your hard work and for new release of nilfs-tools.
I'd like to share two patches (attached) to fix some minor problems that I've
noticed:
* man.patch (with minor man pages corrections)
* fhs.patch (to install nilfs-* executables to "/usr/sbin" instead of "/sbin"
due to library dependencies in "/usr").
Please review and apply (if appropriate).
Also please CC to me as I'm not subscribed.
Thank you.
--
All the best,
Dmitry Smirnov
GPG key : 4096R/53968D1B
[-- Attachment #1.3: fhs.patch --]
[-- Type: text/x-patch, Size: 1803 bytes --]
Last-Update: 2014-04-11
Forwarded: yes
Author: Dmitry Smirnov <onlyjob-IGUgQLVVQiRCV4ILt04nZQ@public.gmane.org>
Description: install nilfs-* executables to /usr/sbin:
Fixes the following problems (thanks, `adequate`):
bin-or-sbin-binary-requires-usr-lib-library
/sbin/nilfs-clean => /usr/lib/x86_64-linux-gnu/libnilfscleaner.so.0
/sbin/nilfs-clean => /usr/lib/x86_64-linux-gnu/libnilfs.so.0
/sbin/nilfs-resize => /usr/lib/x86_64-linux-gnu/libnilfs.so.0
/sbin/nilfs-resize => /usr/lib/x86_64-linux-gnu/libnilfsgc.so.0
/sbin/nilfs-tune => /usr/lib/x86_64-linux-gnu/libnilfs.so.0
/sbin/nilfs_cleanerd => /usr/lib/x86_64-linux-gnu/libnilfs.so.0
/sbin/nilfs_cleanerd => /usr/lib/x86_64-linux-gnu/libnilfsgc.so.0
--- a/configure.ac
+++ b/configure.ac
@@ -165,9 +165,10 @@
AC_SYS_LARGEFILE
# Install directories
AC_PREFIX_DEFAULT([/usr])
-AC_SUBST([sbindir], [/sbin])
+AC_SUBST([root_sbindir], [/sbin])
+AC_SUBST([sbindir], [${prefix}/sbin])
AC_SUBST([sysconfdir], [/etc])
AC_SUBST([localstatedir], [/var])
AC_CONFIG_FILES([Makefile
--- a/sbin/mkfs/Makefile.am
+++ b/sbin/mkfs/Makefile.am
@@ -5,9 +5,9 @@
LDADD = -luuid $(LIB_BLKID) $(top_builddir)/lib/libnilfsfeature.la \
$(top_builddir)/lib/libmountchk.la \
$(top_builddir)/lib/libcrc32.la
-sbin_PROGRAMS = mkfs.nilfs2
+root_sbin_PROGRAMS = mkfs.nilfs2
mkfs_nilfs2_SOURCES = mkfs.c bitops.c mkfs.h
EXTRA_DIST = .gitignore
--- a/sbin/mount/Makefile.am
+++ b/sbin/mount/Makefile.am
@@ -10,9 +10,9 @@
AM_CPPFLAGS = -I$(top_srcdir)/include
LDADD = $(top_builddir)/lib/librealpath.la \
$(top_builddir)/lib/libcleanerexec.la $(LIB_MOUNT) $(LIB_SELINUX)
-sbin_PROGRAMS = mount.nilfs2 umount.nilfs2
+root_sbin_PROGRAMS = mount.nilfs2 umount.nilfs2
if CONFIG_LIBMOUNT
COMMONSOURCES += mount_attrs.c
[-- Attachment #1.4: man.patch --]
[-- Type: text/x-patch, Size: 1195 bytes --]
Last-Update: 2014-04-11
Forwarded: yes
Author: Dmitry Smirnov <onlyjob-IGUgQLVVQiRCV4ILt04nZQ@public.gmane.org>
Description: minor man pages lintianisation
spelling-error-in-manpage
hyphen-used-as-minus-sign
--- a/man/nilfs-clean.8
+++ b/man/nilfs-clean.8
@@ -19,9 +19,9 @@
sent to and performed on the \fBnilfs_cleanerd\fP(8) program. If no
commands are specified, \fBnilfs-clean\fP just triggers a one-pass
cleaning.
.PP
-When \fIdevice\fP is ommitted, \fBnilfs-clean\fP selects an active
+When \fIdevice\fP is omitted, \fBnilfs-clean\fP selects an active
NILFS2 file system in the system.
.PP
This command is valid only for mounted NILFS2 file systems, and
will fail if the \fIdevice\fP has no active mounts.
--- a/man/umount.nilfs2.8
+++ b/man/umount.nilfs2.8
@@ -41,9 +41,9 @@
Verbose mode.
.SH THE LOOP DEVICE
.B umount.nilfs2
will free the loop device (if any) associated with the mount, in case
-it finds the option `loop=...' in \fI/etc/mtab\fP. The -d option,
+it finds the option `loop=...' in \fI/etc/mtab\fP. The \-d option,
which also allows \fBumount\fP(8) to free the loop device without the
`loop=...' option, is not supported.
.SH AUTHOR
.B umount.nilfs2
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
next reply other threads:[~2014-04-11 7:49 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-11 7:49 Dmitry Smirnov [this message]
2014-04-13 21:36 ` few patches from Debian Ryusuke Konishi
[not found] ` <20140414.063654.356443912.konishi.ryusuke-Zyj7fXuS5i5L9jVzuh4AOg@public.gmane.org>
2014-04-16 1:22 ` Dmitry Smirnov
2014-04-19 10:58 ` Ryusuke Konishi
[not found] ` <20140419.195816.57462893.konishi.ryusuke-Zyj7fXuS5i5L9jVzuh4AOg@public.gmane.org>
2014-04-19 11:17 ` Dmitry Smirnov
2014-04-19 11:26 ` Ryusuke Konishi
[not found] ` <20140419.202620.220041591.konishi.ryusuke-Zyj7fXuS5i5L9jVzuh4AOg@public.gmane.org>
2014-04-19 11:46 ` Dmitry Smirnov
2014-04-19 12:00 ` Ryusuke Konishi
[not found] ` <20140419.210014.85398266.konishi.ryusuke-Zyj7fXuS5i5L9jVzuh4AOg@public.gmane.org>
2014-04-19 12:39 ` Dmitry Smirnov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2256155.lyPQvkPv8r@debstor \
--to=onlyjob-8fiuurrzop0dnm+yrofe0a@public.gmane.org \
--cc=linux-nilfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.