You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@zookeeper.apache.org by GitBox <gi...@apache.org> on 2021/07/12 09:04:56 UTC
[GitHub] [zookeeper] bigmarvin opened a new pull request #1726: ZOOKEEPER-4331: add headers back
bigmarvin opened a new pull request #1726:
URL: https://github.com/apache/zookeeper/pull/1726
Following up https://github.com/apache/zookeeper/pull/1720, I'm applying the change on master instead of 3.5.x as advised.
In addition to the original change, pom of zookeeper-jute is also updated because it provides some packages required by zookeeper-server. In this way, it must serve as a bundle directly before zookeeper-server can serve as a bundle.
This remans my preferred approach while I'm preparing the alternatives on master. I'm looking for a fix to 3.6, 3.7 and 3.8, so please let me know if any further PR is required for this to happen.
--
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: notifications-unsubscribe@zookeeper.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [zookeeper] nkalmar closed pull request #1726: ZOOKEEPER-4331: add headers back
Posted by GitBox <gi...@apache.org>.
nkalmar closed pull request #1726:
URL: https://github.com/apache/zookeeper/pull/1726
--
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: notifications-unsubscribe@zookeeper.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [zookeeper] nkalmar edited a comment on pull request #1726: ZOOKEEPER-4331: add headers back
Posted by GitBox <gi...@apache.org>.
nkalmar edited a comment on pull request #1726:
URL: https://github.com/apache/zookeeper/pull/1726#issuecomment-891704792
I believe this is a duplicate of https://github.com/apache/zookeeper/pull/1727
Closing. @bigmarvin , please ping me or @eolivelli if I got something wrong here.
Oh, sorry, not a duplicate but the other one was creating another artifact for osgi, which is the one we wanted. So this is no longer needed I believe.
--
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: notifications-unsubscribe@zookeeper.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [zookeeper] nkalmar commented on pull request #1726: ZOOKEEPER-4331: add headers back
Posted by GitBox <gi...@apache.org>.
nkalmar commented on pull request #1726:
URL: https://github.com/apache/zookeeper/pull/1726#issuecomment-891704792
I believe this is a duplicate of https://github.com/apache/zookeeper/pull/1727
Closing. @bigmarvin , please ping me or @eolivelli if I got something wrong here.
--
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: notifications-unsubscribe@zookeeper.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org