Re: [resource-agents][PATCH] repair wrong path for ocf_root when prefix ne default

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 09/24/2012 04:57 PM, Grüninger, Andreas (LGL Extern) wrote:
> I don't know if this is the right list. If not please advise whhich list I should use.

There is no mailing list for resource-agents.

What you want to do is fork the tree on git hub, publish your changes
and request a pull.

Does the tree pass a make distcheck with this change?

Fabio

> 
> When the resource-agents are configured to use a prefix other than /usr and the initial value for ocf-root is read from {PREFIX}/include/heartbeat/glue_config.h  a wrong path for the the resulting ocf-root is constructed.
> The wrong result begins with {PREFIX}/{PREFIX}/lib/ocf/.
> The inital value read from {PREFIX}/include/heartbeat/glue_config.h contains already an absolute path and the prefix is errounesly used a second time.
> 
> commit 4779b382fb65cc457db2217b713c864388ad62a7
> Author: andreas.grueninger <andreas.grueninger@server>
> Date:   Mon Sep 24 16:35:12 2012 +0200
> 
>     repair wrong path for ocf_root when prefix ne default
> 
> diff --git a/Makefile.am b/Makefile.am
> index 28634c0..242f651 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -50,24 +50,35 @@ EXTRA_DIST		= autogen.sh .version make/release.mk \
>  
>  install-exec-local:
>  if BUILD_LINUX_HA
> -	$(INSTALL) -d -m 1755 $(DESTDIR)$(HA_RSCTMPDIR)
> -	$(LN_S) ../../lib/heartbeat/ocf-binaries $(DESTDIR)${OCF_RA_DIR_PREFIX}/heartbeat/.ocf-binaries
> -	$(LN_S) ../../lib/heartbeat/ocf-directories $(DESTDIR)${OCF_RA_DIR_PREFIX}/heartbeat/.ocf-directories
> -	$(LN_S) ../../lib/heartbeat/ocf-returncodes $(DESTDIR)${OCF_RA_DIR_PREFIX}/heartbeat/.ocf-returncodes
> -	$(LN_S) ../../lib/heartbeat/ocf-shellfuncs $(DESTDIR)${OCF_RA_DIR_PREFIX}/heartbeat/.ocf-shellfuncs
> +	$(INSTALL) -d -m 1755 $(HA_RSCTMPDIR)
> +	$(LN_S) -f ../../lib/heartbeat/ocf-binaries ${OCF_RA_DIR_PREFIX}/heartbeat/.ocf-binaries
> +	$(LN_S) -f ../../lib/heartbeat/ocf-directories ${OCF_RA_DIR_PREFIX}/heartbeat/.ocf-directories
> +	$(LN_S) -f ../../lib/heartbeat/ocf-returncodes ${OCF_RA_DIR_PREFIX}/heartbeat/.ocf-returncodes
> +	$(LN_S) -f ../../lib/heartbeat/ocf-shellfuncs ${OCF_RA_DIR_PREFIX}/heartbeat/.ocf-shellfuncs
>  endif
>  if BUILD_RGMANAGER
>  if BUILD_LINUX_HA
> -	$(LN_S) ${CLUSTERDATA} $(DESTDIR)${OCF_RA_DIR_PREFIX}/redhat
> +	$(LN_S) ${CLUSTERDATA} ${OCF_RA_DIR_PREFIX}/redhat
>  endif
> -	$(INSTALL) -d $(DESTDIR)/$(LOGDIR)
> +	$(INSTALL) -d $(LOGDIR)
>  endif
>  
>  dist-clean-local:
>  	rm -f autoconf automake autoheader $(TARFILES)
>  
>  uninstall-local:
> -			rmdir $(DESTDIR)/$(LOGDIR) || :;
> +if BUILD_LINUX_HA
> +	rm -f ${OCF_RA_DIR_PREFIX}/heartbeat/.ocf-binaries
> +	rm -f ${OCF_RA_DIR_PREFIX}/heartbeat/.ocf-directories
> +	rm -f ${OCF_RA_DIR_PREFIX}/heartbeat/.ocf-returncodes
> +	rm -f ${OCF_RA_DIR_PREFIX}/heartbeat/.ocf-shellfuncs
> +endif
> +if BUILD_RGMANAGER
> +if BUILD_LINUX_HA
> +	rm -f ${OCF_RA_DIR_PREFIX}/redhat
> +endif
> +	rm -f $(LOGDIR) || :;    
> +endif
>  
>  BUILT_SOURCES = .version
>  .version:
> diff --git a/configure.ac b/configure.ac
> index 456e901..4fef729 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -154,13 +154,11 @@ AC_ARG_WITH(initdir,
>  OCF_ROOT_DIR="/usr/lib/ocf"
>  AC_ARG_WITH(ocf-root,
>      [  --with-ocf-root=DIR      directory for OCF scripts [${OCF_ROOT_DIR}]],
> -    [ if test x"$withval" = xprefix; then OCF_ROOT_DIR=${prefix}; else
> -	 OCF_ROOT_DIR="$withval"; fi ])
> +	 OCF_ROOT_DIR="$withval" ])
>  HA_RSCTMPDIR=${localstatedir}/run/resource-agents
>  AC_ARG_WITH(rsctmpdir,
>      [  --with-rsctmpdir=DIR      directory for resource agents state files [${HA_RSCTMPDIR}]],
> -    [ if test x"$withval" = xprefix; then HA_RSCTMPDIR=${prefix}; else
> -	 HA_RSCTMPDIR="$withval"; fi ])
> +	 HA_RSCTMPDIR="$withval" ])
>  
>  AC_ARG_ENABLE([libnet],
>   [  --enable-libnet 	Use libnet for ARP based funcationality, [default=try]], 
> @@ -393,26 +391,18 @@ HA_VARLIBHBDIR=${localstatedir}/lib/heartbeat
>  AC_DEFINE_UNQUOTED(HA_VARLIBHBDIR,"$HA_VARLIBHBDIR", Whatever this used to mean)
>  AC_SUBST(HA_VARLIBHBDIR)
>  
> -OCF_RA_DIR="${OCF_ROOT_DIR}/resource.d/"
> +OCF_RA_DIR="${OCF_ROOT_DIR}/resource.d"
>  AC_DEFINE_UNQUOTED(OCF_RA_DIR,"$OCF_RA_DIR", Location for OCF RAs)
>  AC_SUBST(OCF_RA_DIR)
>  
> -if test "${prefix}" = "/usr"; then
> - OCF_RA_DIR_PREFIX="$OCF_RA_DIR"
> -else
> - OCF_RA_DIR_PREFIX="${prefix}/$OCF_RA_DIR"
> -fi
> +OCF_RA_DIR_PREFIX="$OCF_RA_DIR"
>  AC_SUBST(OCF_RA_DIR_PREFIX)
>  
> -OCF_LIB_DIR="${OCF_ROOT_DIR}/lib/"
> +OCF_LIB_DIR="${OCF_ROOT_DIR}/lib"
>  AC_DEFINE_UNQUOTED(OCF_LIB_DIR,"$OCF_LIB_DIR", Location for shared code for OCF RAs)
>  AC_SUBST(OCF_LIB_DIR)
>  
> -if test "${prefix}" = "/usr"; then
> - OCF_LIB_DIR_PREFIX="$OCF_LIB_DIR"
> -else
> - OCF_LIB_DIR_PREFIX="${prefix}/$OCF_LIB_DIR"
> -fi
> +OCF_LIB_DIR_PREFIX="$OCF_LIB_DIR"
>  AC_SUBST(OCF_LIB_DIR_PREFIX)
>  
>  dnl ===============================================
> 
> _______________________________________________
> discuss mailing list
> discuss@xxxxxxxxxxxx
> http://lists.corosync.org/mailman/listinfo/discuss
> 

_______________________________________________
discuss mailing list
discuss@xxxxxxxxxxxx
http://lists.corosync.org/mailman/listinfo/discuss



[Index of Archives]     [Linux Clusters]     [Corosync Project]     [Linux USB Devel]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Linux Kernel]     [Linux SCSI]     [X.Org]

  Powered by Linux