| Message ID | 20180717101244.GC11106@onega.vda.li (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Perfect :)
Reviewed-by: Aurelien Aptel <aaptel@suse.com>
вт, 17 июл. 2018 г. в 4:21, Aurélien Aptel <aaptel@samba.org>: > > Perfect :) > > Reviewed-by: Aurelien Aptel <aaptel@suse.com> > > -- > Aurélien Aptel / SUSE Labs Samba Team > GPG: 1839 CB5F 9F5B FB9B AA97 8C99 03C8 A49B 521B D5D3 > SUSE Linux GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany > GF: Felix Imendörffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nürnberg) > -- > To unsubscribe from this list: send the line "unsubscribe linux-cifs" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html Merged into next. -- Best regards, Pavel Shilovsky
On 2018-07-17, Alexander Bokovoy said: > > Right. I replaced rst2man in RST2MAN definition in Makefile.am by > $(have_rst2man) and given that we only call for that when CONFIG_MAN > is set, that should complete the change. ... >+ AC_CHECK_PROGS(have_rst2man, rst2man-3.6 rst2man-3.4 rst2man-3 rst2man, no) Some OS's (Gentoo Linux, possibly OSX?) install rst2man as rst2man.py. Please do: - AC_CHECK_PROGS(have_rst2man, rst2man-3.6 rst2man-3.4 rst2man-3 rst2man, no) + AC_CHECK_PROGS(have_rst2man, rst2man-3.6 rst2man-3.4 rst2man-3 rst2man.py rst2man, no) * I couldn't find a git repo where the above had actually landed? https://wiki.samba.org/index.php/LinuxCIFS points to https://wiki.samba.org/index.php/LinuxCIFS_utils, which points to git://git.samba.org/cifs-utils.git, which has not gotten a commit since 2018-07-10 (master) or 2018-07-15 (next), unless I am just dumb. Thanks,
пн, 11 февр. 2019 г. в 17:22, Hank Leininger <hlein@korelogic.com>: > > On 2018-07-17, Alexander Bokovoy said: > > > > Right. I replaced rst2man in RST2MAN definition in Makefile.am by > > $(have_rst2man) and given that we only call for that when CONFIG_MAN > > is set, that should complete the change. > ... > >+ AC_CHECK_PROGS(have_rst2man, rst2man-3.6 rst2man-3.4 rst2man-3 rst2man, no) > > Some OS's (Gentoo Linux, possibly OSX?) install rst2man as rst2man.py. > > Please do: > > - AC_CHECK_PROGS(have_rst2man, rst2man-3.6 rst2man-3.4 rst2man-3 rst2man, no) > + AC_CHECK_PROGS(have_rst2man, rst2man-3.6 rst2man-3.4 rst2man-3 rst2man.py rst2man, no) > > * I couldn't find a git repo where the above had actually landed? > https://wiki.samba.org/index.php/LinuxCIFS points to > https://wiki.samba.org/index.php/LinuxCIFS_utils, which points to > git://git.samba.org/cifs-utils.git, which has not gotten a commit > since 2018-07-10 (master) or 2018-07-15 (next), unless I am just dumb. > Hi Hank, The repository is currently here: https://github.com/piastry/cifs-utils/commits/next The master branch on git.samba.org will be updated once I make a release. Feel free to submit a patch on top of the "next" branch. -- Best regards, Pavel Shilovsky
On Mon, Feb 11, 2019 at 05:36:13PM -0800, Pavel Shilovsky wrote: > ????, 11 ????????. 2019 ??. ?? 17:22, Hank Leininger <hlein@korelogic.com>: > > On 2018-07-17, Alexander Bokovoy said: > > >+ AC_CHECK_PROGS(have_rst2man, rst2man-3.6 rst2man-3.4 rst2man-3 rst2man, no) > > > > Some OS's (Gentoo Linux, possibly OSX?) install rst2man as rst2man.py. > > > > Please do: > > > > - AC_CHECK_PROGS(have_rst2man, rst2man-3.6 rst2man-3.4 rst2man-3 rst2man, no) > > + AC_CHECK_PROGS(have_rst2man, rst2man-3.6 rst2man-3.4 rst2man-3 rst2man.py rst2man, no) > > > > * I couldn't find a git repo where the above had actually landed? > The repository is currently here: > > https://github.com/piastry/cifs-utils/commits/next Thanks! Just submitted a PR there. Hank
Hi Pavel, Pavel Shilovsky <piastryyy@gmail.com> writes: > The repository is currently here: > > https://github.com/piastry/cifs-utils/commits/next > > The master branch on git.samba.org will be updated once I make a > release. Feel free to submit a patch on top of the "next" branch. Do you prefer email patches here or github PR? I have sent a cifs-utils patch some ago to correct the kernel release version in the smbinfo man page. https://patchwork.kernel.org/patch/10779659/ Cheers,
вт, 12 февр. 2019 г. в 02:42, Aurélien Aptel <aaptel@suse.com>: > > Hi Pavel, > > Pavel Shilovsky <piastryyy@gmail.com> writes: > > The repository is currently here: > > > > https://github.com/piastry/cifs-utils/commits/next > > > > The master branch on git.samba.org will be updated once I make a > > release. Feel free to submit a patch on top of the "next" branch. > > Do you prefer email patches here or github PR? I have sent a cifs-utils > patch some ago to correct the kernel release version in the smbinfo man > page. > > https://patchwork.kernel.org/patch/10779659/ The patch has been merged together with other pending ones. Feel free to to use github PRs in future but patches are ok too. -- Best regards, Pavel Shilovsky
diff --git a/Makefile.am b/Makefile.am index 30658e3..f37c9ae 100644 --- a/Makefile.am +++ b/Makefile.am @@ -16,7 +16,7 @@ man_MANS= SUFFIXES = .rst .1 .8 -RST2MAN = rst2man --syntax-highlight=none $< $@ +RST2MAN = $(have_rst2man) --syntax-highlight=none $< $@ .rst.1: $(RST2MAN) diff --git a/configure.ac b/configure.ac index b0bc2b9..8e3d6ce 100644 --- a/configure.ac +++ b/configure.ac @@ -252,12 +252,12 @@ fi # if docs are not disabled, check if rst2man is available if test $enable_man != "no"; then - AC_CHECK_PROG(have_rst2man, rst2man, yes, no) + AC_CHECK_PROGS(have_rst2man, rst2man-3.6 rst2man-3.4 rst2man-3 rst2man, no) if test $have_rst2man = "no"; then if test $enable_man = "yes"; then - AC_MSG_ERROR([rst2man not found: cannot generate man pages, consider installing perl.]) + AC_MSG_ERROR([rst2man not found: cannot generate man pages, consider installing python{2,3}-docutils.]) else - AC_MSG_WARN([rst2man not found: cannot generate man pages, consider installing perl. Disabling man page generation.]) + AC_MSG_WARN([rst2man not found: cannot generate man pages, consider installing python{2,3}-docutils. Disabling man page generation.]) enable_man="no" fi else