[PATCH cygport] rubygems.cygclass: set relative paths in bindir

[PATCH cygport] rubygems.cygclass: set relative paths in bindir

[Daisuke Fujimura]

[-- Attachment #1: Type: text/plain, Size: 351 bytes --]

If the `--build-root` option is specified for the `gem` command, the
value of the `--bindir` option must be a relative path.

https://github.com/rubygems/rubygems/blob/v3.4.12/lib/rubygems/installer.rb#L678-L679

This problem seems to have occurred since this commit.

https://github.com/cygwin/cygport/commit/18bd795f82d3abeff333d35ce089dd55fa2b192c

[-- Attachment #2: rubygems.cygclass.patch --]
[-- Type: application/octet-stream, Size: 686 bytes --]

diff --git a/cygclass/rubygem.cygclass b/cygclass/rubygem.cygclass
index 13d7b1c5..4c6b710d 100644
--- a/cygclass/rubygem.cygclass
+++ b/cygclass/rubygem.cygclass
@@ -132,7 +132,7 @@ rubygem_install() {
 #****
 	CONFIGURE_ARGS="--with-cflags='${CFLAGS} -fdebug-prefix-map=${D}${RUBYGEM_INSTDIR}=/usr/src/debug/${NAME}-${VERSION}-${RELEASE}' $RUBYGEM_CONFIGURE_ARGS $@" \
 	gem install -V --local --ignore-dependencies --document=rdoc,ri \
-		--build-root ${D} --install-dir ${RUBYGEM_DIR} --bindir ${D}$(__host_prefix)/bin \
+		--build-root ${D} --install-dir ${RUBYGEM_DIR} --bindir $(__host_prefix)/bin \
 		${RUBYGEM_NAME}-${VERSION}.gem
 
 	# used later to clean up RUBYGEM_INSTDIR

Re: [PATCH cygport] rubygems.cygclass: set relative paths in bindir

[Jon Turney]

On 02/05/2023 11:16, Daisuke Fujimura via Cygwin-apps wrote:
> If the `--build-root` option is specified for the `gem` command, the
> value of the `--bindir` option must be a relative path.
> 
> https://github.com/rubygems/rubygems/blob/v3.4.12/lib/rubygems/installer.rb#L678-L679
> 
> This problem seems to have occurred since this commit.
> 
> https://github.com/cygwin/cygport/commit/18bd795f82d3abeff333d35ce089dd55fa2b192c

Thanks. Applied.

I also tweaked rubygem_install() so 'gem install' failing causes an 
error, rather than continuing on regardless.