Skip to content

Conversation

@crsqq
Copy link

@crsqq crsqq commented Sep 4, 2024

Fixed the following error when building the Docker image:

STEP 4/8: RUN pip git+https://github.com/zangobot/ember.git
ERROR: unknown command "git+https://github.com/zangobot/ember.git" 
Error: building at STEP "RUN pip git+https://github.com/zangobot/ember.git": while running runtime: exit status 1

@zangobot
Copy link
Collaborator

zangobot commented Sep 5, 2024

Thanks for the issue, but that repository is needed.
I will fix it soon.

@crsqq
Copy link
Author

crsqq commented Sep 5, 2024

Appreciate the fast response.
https://github.com/zangobot/ember.git is already included in the requirements.txt, therefore removing it from the Dockerfile should not have any negative effect.

@zangobot
Copy link
Collaborator

Did you test this befire / after the modification to check whether it still works inside the container?

@crsqq
Copy link
Author

crsqq commented Feb 12, 2025

Yes I did.

Somehow tqdm was removed from requirements.txt in the meantime, which results in an error:

$ docker run --rm -it secml_malware:0.3.2 python -c "import secml_malware; print(secml_malware.__version__)"

...

Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/ramen/secml_malware/secml_malware/__init__.py", line 5, in <module>
    from secml_malware import attack, models, utils, visualization
  File "/ramen/secml_malware/secml_malware/models/__init__.py", line 4, in <module>
    from .c_classifier_ember import CClassifierEmber
  File "/ramen/secml_malware/secml_malware/models/c_classifier_ember.py", line 3, in <module>
    from ember import PEFeatureExtractor
  File "/usr/local/lib/python3.9/site-packages/ember/__init__.py", line 5, in <module>
    import tqdm
ModuleNotFoundError: No module named 'tqdm

With changes to requirements.txt:

docker run --rm -it secml_malware:0.3.2 python -c "import secml_malware; print(secml_malware.__version__)"
...
0.3.2

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.

2 participants