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

rsa icons #237

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

rsa icons #237

wants to merge 2 commits into from

Conversation

caspoone
Copy link

@caspoone caspoone commented Jul 15, 2020

Added various RSA NetWitness icons to resources.
Issue #: 219

@mingrammer
Copy link
Owner

Please run the autogen.sh script to add new nodes.

This is contribution guideline: https://github.com/mingrammer/diagrams/blob/master/CONTRIBUTING.md

@mingrammer mingrammer added comp/node Issue of node component kind/feat/icon Icon request area/provider/onprem On-Premise provider labels Jul 16, 2020
@caspoone
Copy link
Author

Please run the autogen.sh script to add new nodes.

This is contribution guideline: https://github.com/mingrammer/diagrams/blob/master/CONTRIBUTING.md

Hello, I am running into an issue running the script. Any thoughts?

Screenshot from 2020-07-16 09-59-35

@mingrammer
Copy link
Owner

You must use Python >= 3.6. You’re using 2.7 version.

@caspoone
Copy link
Author

I have ran the autogen script as requested 😃

@leandrodamascena
Copy link
Contributor

@caspoone you should not modify "autogen.sh" to execute python3 instead python. Python3 alias is specific and by default python is a convencional name.
I'll add a PR to check python version before execute autogen.
Please, also update conflicts..

@mingrammer
Copy link
Owner

Any progress? :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/provider/onprem On-Premise provider comp/node Issue of node component kind/feat/icon Icon request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants