diff mbox

[1/1] mono: remove copy of host etc files on install

Message ID 20170315022458.14160-1-dustin.r.johnson@gmail.com
State Accepted
Headers show

Commit Message

Dustin Johnson March 15, 2017, 2:24 a.m. UTC
When the mono package is installed, the autotools installer installs
the /etc/mono files to the target. A post_install hook then copies
over the mono libraries to the target as well as the host /etc/mono
files which overrides the target files. The target specific mono
configuration file (/etc/mono/config) is overridden with the host
settings. This causes mono on the target to be unable to locate target
specific .so files as it overrides the changes enacted by the patches
for the package.

Signed-off-by: Dustin Johnson <dustin.r.johnson@gmail.com>
---
 package/mono/mono.mk | 1 -
 1 file changed, 1 deletion(-)

Comments

Thomas Petazzoni March 18, 2017, 2:39 p.m. UTC | #1
Hello,

Angelo, could you comment on this patch touching the Mono package?

Copying $(HOST_DIR)/etc/mono to $(TARGET_DIR)/etc has been there since
you contributed the package, but apparently causes some issues. It
would be nice if you could have a look at the patch proposed by Dustin.

Thanks a lot!

Thomas

On Tue, 14 Mar 2017 22:24:58 -0400, Dustin Johnson wrote:
> When the mono package is installed, the autotools installer installs
> the /etc/mono files to the target. A post_install hook then copies
> over the mono libraries to the target as well as the host /etc/mono
> files which overrides the target files. The target specific mono
> configuration file (/etc/mono/config) is overridden with the host
> settings. This causes mono on the target to be unable to locate target
> specific .so files as it overrides the changes enacted by the patches
> for the package.
> 
> Signed-off-by: Dustin Johnson <dustin.r.johnson@gmail.com>
> ---
>  package/mono/mono.mk | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/package/mono/mono.mk b/package/mono/mono.mk
> index 20b920568..4996b7c60 100644
> --- a/package/mono/mono.mk
> +++ b/package/mono/mono.mk
> @@ -31,7 +31,6 @@ MONO_CONF_OPTS = --disable-gtk-doc \
>  define MONO_INSTALL_LIBS
>  	rsync -av --exclude=*.so --exclude=*.mdb \
>  		$(HOST_DIR)/usr/lib/mono $(TARGET_DIR)/usr/lib/
> -	rsync -av $(HOST_DIR)/etc/mono $(TARGET_DIR)/etc
>  endef
>  
>  MONO_POST_INSTALL_TARGET_HOOKS += MONO_INSTALL_LIBS
Angelo Compagnucci May 22, 2017, 2:52 p.m. UTC | #2
2017-03-15 3:24 GMT+01:00 Dustin Johnson <dustin.r.johnson@gmail.com>:
> When the mono package is installed, the autotools installer installs
> the /etc/mono files to the target. A post_install hook then copies
> over the mono libraries to the target as well as the host /etc/mono
> files which overrides the target files. The target specific mono
> configuration file (/etc/mono/config) is overridden with the host
> settings. This causes mono on the target to be unable to locate target
> specific .so files as it overrides the changes enacted by the patches
> for the package.
>
> Signed-off-by: Dustin Johnson <dustin.r.johnson@gmail.com>
Tested-by: Angelo Compagnucci <angelo.compagnucci@gmail.com>
Reviewed-by: Angelo Compagnucci <angelo.compagnucci@gmail.com>
Acked-by: Angelo Compagnucci <angelo.compagnucci@gmail.com>
> ---
>  package/mono/mono.mk | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/package/mono/mono.mk b/package/mono/mono.mk
> index 20b920568..4996b7c60 100644
> --- a/package/mono/mono.mk
> +++ b/package/mono/mono.mk
> @@ -31,7 +31,6 @@ MONO_CONF_OPTS = --disable-gtk-doc \
>  define MONO_INSTALL_LIBS
>         rsync -av --exclude=*.so --exclude=*.mdb \
>                 $(HOST_DIR)/usr/lib/mono $(TARGET_DIR)/usr/lib/
> -       rsync -av $(HOST_DIR)/etc/mono $(TARGET_DIR)/etc
>  endef
>
>  MONO_POST_INSTALL_TARGET_HOOKS += MONO_INSTALL_LIBS
> --
> 2.12.0.windows.1
>
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
Thomas Petazzoni May 23, 2017, 2:29 p.m. UTC | #3
Hello,

On Tue, 14 Mar 2017 22:24:58 -0400, Dustin Johnson wrote:
> When the mono package is installed, the autotools installer installs
> the /etc/mono files to the target. A post_install hook then copies
> over the mono libraries to the target as well as the host /etc/mono
> files which overrides the target files. The target specific mono
> configuration file (/etc/mono/config) is overridden with the host
> settings. This causes mono on the target to be unable to locate target
> specific .so files as it overrides the changes enacted by the patches
> for the package.
> 
> Signed-off-by: Dustin Johnson <dustin.r.johnson@gmail.com>
> ---
>  package/mono/mono.mk | 1 -
>  1 file changed, 1 deletion(-)

Applied to master, thanks. Thanks also to Angelo for having
tested/reviewed the patch!

Best regards,

Thomas
Peter Korsgaard June 1, 2017, 2:15 p.m. UTC | #4
>>>>> "Angelo" == Angelo Compagnucci <angelo.compagnucci@gmail.com> writes:

 > 2017-03-15 3:24 GMT+01:00 Dustin Johnson <dustin.r.johnson@gmail.com>:
 >> When the mono package is installed, the autotools installer installs
 >> the /etc/mono files to the target. A post_install hook then copies
 >> over the mono libraries to the target as well as the host /etc/mono
 >> files which overrides the target files. The target specific mono
 >> configuration file (/etc/mono/config) is overridden with the host
 >> settings. This causes mono on the target to be unable to locate target
 >> specific .so files as it overrides the changes enacted by the patches
 >> for the package.
 >> 
 >> Signed-off-by: Dustin Johnson <dustin.r.johnson@gmail.com>
 > Tested-by: Angelo Compagnucci <angelo.compagnucci@gmail.com>
 > Reviewed-by: Angelo Compagnucci <angelo.compagnucci@gmail.com>
 > Acked-by: Angelo Compagnucci <angelo.compagnucci@gmail.com>

Committed to 2017.02.x, thanks.
diff mbox

Patch

diff --git a/package/mono/mono.mk b/package/mono/mono.mk
index 20b920568..4996b7c60 100644
--- a/package/mono/mono.mk
+++ b/package/mono/mono.mk
@@ -31,7 +31,6 @@  MONO_CONF_OPTS = --disable-gtk-doc \
 define MONO_INSTALL_LIBS
 	rsync -av --exclude=*.so --exclude=*.mdb \
 		$(HOST_DIR)/usr/lib/mono $(TARGET_DIR)/usr/lib/
-	rsync -av $(HOST_DIR)/etc/mono $(TARGET_DIR)/etc
 endef
 
 MONO_POST_INSTALL_TARGET_HOOKS += MONO_INSTALL_LIBS