You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bookkeeper.apache.org by "shoothzj (via GitHub)" <gi...@apache.org> on 2024/07/28 12:57:20 UTC

[PR] fix: install netcat-openbsd instead of netcat in test image build [bookkeeper]

shoothzj opened a new pull request, #4476:
URL: https://github.com/apache/bookkeeper/pull/4476

   (no comment)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@bookkeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix: install netcat-openbsd instead of netcat in test image build [bookkeeper]

Posted by "shoothzj (via GitHub)" <gi...@apache.org>.
shoothzj commented on PR #4476:
URL: https://github.com/apache/bookkeeper/pull/4476#issuecomment-2254916439

   @lhotari I change `FROM eclipse-temurin:8-jdk` to `FROM eclipse-temurin:8-jdk-jammy`, to keep style consistent with other ubuntu docker image.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@bookkeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix: install netcat-openbsd instead of netcat in test image build [bookkeeper]

Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on PR #4476:
URL: https://github.com/apache/bookkeeper/pull/4476#issuecomment-2254580927

   ```
    The command '/bin/sh -c wget -nv https://archive.apache.org/dist/zookeeper/bookkeeper/bookkeeper-4.1.0/bookkeeper-server-4.1.0-bin.tar.gz{,.sha1,.md5,.asc}' returned a non-zero code: 8
   ```
   
   It looks like the command doesn't get properly expanded. I wonder if using bash shell would address the problem.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@bookkeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix: install netcat-openbsd instead of netcat in test image build [bookkeeper]

Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on PR #4476:
URL: https://github.com/apache/bookkeeper/pull/4476#issuecomment-2254970184

   > @lhotari I change `FROM eclipse-temurin:8-jdk` to `FROM eclipse-temurin:8-jdk-jammy`, to keep style consistent with other ubuntu docker image.
   
   Oh yes, makes sense. It seems that things broken when eclipse-temurin switched to use Ubuntu 24.04 (noble) base image. Switching to jammy (22.04) makes the existing commands to work.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@bookkeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix: install netcat-openbsd instead of netcat in test image build [bookkeeper]

Posted by "shoothzj (via GitHub)" <gi...@apache.org>.
shoothzj merged PR #4476:
URL: https://github.com/apache/bookkeeper/pull/4476


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@bookkeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org