Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Linux-HA: Dev

patch for configure.ac to validate setproctitle

 

 

Linux-HA dev RSS feed   Index | Next | Previous | View Threaded


iidayuus at intellilink

Oct 29, 2009, 10:08 PM

Post #1 of 4 (902 views)
Permalink
patch for configure.ac to validate setproctitle

I found that setproctitle did not become effective.
I made patch of configure.ac to validate setproctitle.
An object is Reusable Cluster Components. [changeset:2194:14d81e757e41]

Best Regards,
Yuusuke IIDA
Attachments: setproctitle_bugfix.patch (2.65 KB)


dejanmm at fastmail

Nov 5, 2009, 2:27 AM

Post #2 of 4 (787 views)
Permalink
Re: patch for configure.ac to validate setproctitle [In reply to]

Hi Yuusuke-sa,

On Fri, Oct 30, 2009 at 02:08:40PM +0900, 飯田 雄介 wrote:
> I found that setproctitle did not become effective.
> I made patch of configure.ac to validate setproctitle.
> An object is Reusable Cluster Components. [changeset:2194:14d81e757e41]

Many thanks for the patch and sorry for the delay. I'll apply it
to the repository after a review.

Cheers,

Dejan

> Best Regards,
> Yuusuke IIDA

> diff -urN glue/configure.ac glue.mod/configure.ac
> --- glue/configure.ac 2009-10-30 11:12:21.000000000 +0900
> +++ glue.mod/configure.ac 2009-10-30 11:14:23.000000000 +0900
> @@ -786,6 +786,77 @@
> AC_CHECK_FUNCS(getopt, AC_DEFINE(HAVE_DECL_GETOPT, 1, [Have getopt function]))
> AC_CHECK_FUNCS(getpeereid)
>
> +dnl **********************************************************************
> +dnl Check for various argv[] replacing functions on various OSs
> +dnl
> +dnl Borrowed from Proftpd
> +dnl Proftpd is Licenced under the terms of the GNU General Public Licence
> +dnl and is available from http://www.proftpd.org/
> +dnl
> +
> +AC_CHECK_FUNCS(setproctitle)
> +AC_CHECK_HEADERS(libutil.h)
> +AC_CHECK_LIB(util, setproctitle,
> + [AC_DEFINE(HAVE_SETPROCTITLE,1,[ ])
> + ac_cv_func_setproctitle="yes" ; LIBS="$LIBS -lutil"])
> +
> +if test "$ac_cv_func_setproctitle" = "yes"; then
> + pf_argv_set="PF_ARGV_NONE"
> +fi
> +
> +if test "$pf_argv_set" = ""; then
> + AC_CHECK_HEADERS(sys/pstat.h)
> + if test "$ac_cv_header_pstat_h" = "yes"; then
> + AC_CHECK_FUNCS(pstat)
> +
> + if test "$ac_cv_func_pstat" = "yes"; then
> + pf_argv_set="PF_ARGV_PSTAT"
> + else
> + pf_argv_set="PF_ARGV_WRITEABLE"
> + fi
> + fi
> +
> + if test "$pf_argv_set" = ""; then
> + AC_EGREP_HEADER([#define.*PS_STRINGS.*],sys/exec.h,
> + have_psstrings="yes",have_psstrings="no")
> + if test "$have_psstrings" = "yes"; then
> + pf_argv_set="PF_ARGV_PSSTRINGS"
> + fi
> + fi
> +
> + if test "$pf_argv_set" = ""; then
> + AC_CACHE_CHECK(whether __progname and __progname_full are available,
> + pf_cv_var_progname,
> + AC_TRY_LINK([extern char *__progname, *__progname_full;],
> + [__progname = "foo"; __progname_full = "foo bar";],
> + pf_cv_var_progname="yes", pf_cv_var_progname="no"))
> +
> + if test "$pf_cv_var_progname" = "yes"; then
> + AC_DEFINE(HAVE___PROGNAME,1,[ ])
> + fi
> +
> + AC_CACHE_CHECK(which argv replacement method to use,
> + pf_cv_argv_type,
> + AC_EGREP_CPP(yes,[.
> +#if defined(__GNU_HURD__)
> + yes
> +#endif
> + ],pf_cv_argv_type="new", pf_cv_argv_type="writeable"))
> +
> + if test "$pf_cv_argv_type" = "new"; then
> + pf_argv_set="PF_ARGV_NEW"
> + fi
> +
> + if test "$pf_argv_set" = ""; then
> + pf_argv_set="PF_ARGV_WRITEABLE"
> + fi
> + fi
> +fi
> +AC_DEFINE_UNQUOTED(PF_ARGV_TYPE, $pf_argv_set,
> + mechanism to pretty-print ps output: setproctitle-equivalent)
> +
> +dnl End of tests borrowed from Proftpd
> +
> dnl ========================================================================
> dnl ltdl
> dnl ========================================================================

> _______________________________________________________
> Linux-HA-Dev: Linux-HA-Dev [at] lists
> http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> Home Page: http://linux-ha.org/

_______________________________________________________
Linux-HA-Dev: Linux-HA-Dev [at] lists
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/


dejanmm at fastmail

Nov 16, 2009, 10:08 AM

Post #3 of 4 (699 views)
Permalink
Re: patch for configure.ac to validate setproctitle [In reply to]

Hi Yuusuke-san,

On Fri, Oct 30, 2009 at 02:08:40PM +0900, 飯田 雄介 wrote:
> I found that setproctitle did not become effective.
> I made patch of configure.ac to validate setproctitle.
> An object is Reusable Cluster Components. [changeset:2194:14d81e757e41]

Patch applied.

Cheers,

Dejan

> Best Regards,
> Yuusuke IIDA
_______________________________________________________
Linux-HA-Dev: Linux-HA-Dev [at] lists
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/


iidayuus at intellilink

Nov 16, 2009, 4:53 PM

Post #4 of 4 (697 views)
Permalink
Re: patch for configure.ac to validate setproctitle [In reply to]

Hi Dejan,

I confirmed an application to the repository.

Thanks,

Yuusuke

Dejan Muhamedagic さんは書きました:
> Hi Yuusuke-san,
>
> On Fri, Oct 30, 2009 at 02:08:40PM +0900, 飯田 雄介 wrote:
>> I found that setproctitle did not become effective.
>> I made patch of configure.ac to validate setproctitle.
>> An object is Reusable Cluster Components. [changeset:2194:14d81e757e41]
>
> Patch applied.
>
> Cheers,
>
> Dejan
>
>> Best Regards,
>> Yuusuke IIDA
> _______________________________________________________
> Linux-HA-Dev: Linux-HA-Dev [at] lists
> http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> Home Page: http://linux-ha.org/

--
----------------------------------------
株式会社メトロシステムズ

飯田雄介
Mail:iidayuus [at] intellilink
----------------------------------------

_______________________________________________________
Linux-HA-Dev: Linux-HA-Dev [at] lists
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/

Linux-HA dev RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.