Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Subsystems for the ragel and cloc binaries #5517

Merged
merged 3 commits into from Feb 26, 2018

Conversation

benjyw
Copy link
Sponsor Contributor

@benjyw benjyw commented Feb 25, 2018

Using the new binary tools mechanism.

Also renames protobuf/subsystem -> protobuf/subsystems, for consistency.

There are two commits, one per subsystem, that may be conveniently reviewed separately.

Also renames protobuf/subsystem -> protobuf/subsystems, for consistency.
# coding=utf-8
# Copyright 2018 Pants project contributors (see CONTRIBUTORS.md).
# Licensed under the Apache License, Version 2.0 (see LICENSE).

Copy link
Sponsor Member

Choose a reason for hiding this comment

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

Should this file be empty? Ditto the other __init__.py

Copy link
Sponsor Contributor Author

Choose a reason for hiding this comment

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

D'oh! That's what I get for letting my IDE create files.

@@ -17,6 +17,8 @@


class BuildozerBinary(NativeTool):
# Note: Not in scope 'buildozer' because that's the name of the singleton task
Copy link
Sponsor Member

Choose a reason for hiding this comment

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

@benjyw benjyw merged commit 114f2ff into pantsbuild:master Feb 26, 2018
@benjyw benjyw deleted the ragel_binary branch February 26, 2018 05:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants