Skip to content

Comments

Revert "defaultGemConfig: remove asciidoctor-diagram JARs"#81737

Merged
zimbatm merged 1 commit intoNixOS:masterfrom
alyssais:asciidoctor
Mar 4, 2020
Merged

Revert "defaultGemConfig: remove asciidoctor-diagram JARs"#81737
zimbatm merged 1 commit intoNixOS:masterfrom
alyssais:asciidoctor

Conversation

@alyssais
Copy link
Member

@alyssais alyssais commented Mar 4, 2020

This reverts commit 1ac11cc.

asciidoctor-diagram starts Java processes, so the JARs are necessary
on all platforms.

See #77149 (comment).

@nomeata please test

This reverts commit 1ac11cc.

asciidoctor-diagram starts Java processes, so the JARs are necessary
on all platforms.

See NixOS#77149 (comment).
@alyssais alyssais added the 2.status: backlog This is a low priority label Mar 4, 2020
@alyssais alyssais requested review from nomeata and zimbatm March 4, 2020 16:44
@alyssais alyssais added 9.needs: port to stable A PR needs a backport to the stable release. and removed 2.status: backlog This is a low priority labels Mar 4, 2020
@ofborg ofborg bot added the 6.topic: ruby A dynamic, open source programming language with a focus on simplicity and productivity. label Mar 4, 2020
Copy link
Contributor

@nomeata nomeata left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, brief local testing successful.

@ofborg ofborg bot added 10.rebuild-darwin: 11-100 This PR causes between 11 and 100 packages to rebuild on Darwin. 10.rebuild-linux: 101-500 This PR causes between 101 and 500 packages to rebuild on Linux. labels Mar 4, 2020
@zimbatm zimbatm merged commit 89a0945 into NixOS:master Mar 4, 2020
zimbatm added a commit that referenced this pull request Mar 4, 2020
This reverts commit 1ac11cc.

asciidoctor-diagram starts Java processes, so the JARs are necessary
on all platforms.

See #77149 (comment).

(cherry picked from commit 89a0945)
@zimbatm
Copy link
Member

zimbatm commented Mar 4, 2020

also cherry-picked into 20.03

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: ruby A dynamic, open source programming language with a focus on simplicity and productivity. 9.needs: port to stable A PR needs a backport to the stable release. 10.rebuild-darwin: 11-100 This PR causes between 11 and 100 packages to rebuild on Darwin. 10.rebuild-linux: 101-500 This PR causes between 101 and 500 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants