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

fix: make pip package actually work #199

Merged
merged 2 commits into from
Dec 20, 2024

Conversation

carlosmmatos
Copy link
Contributor

This PR introduces the ability to install falcon-integration-gateway from the pypi repository and actually use it. A couple key changes were needed for this to work:

  1. Needed to update the setup.py file to include the relevant directories that make the application work (ie fig/ config/)
  2. Updated the config class to check the package level for the defaults.ini file as the fig needs this to instantiate
  3. Updated the config class to allow using local config files for overrides. Now you can pip install falcon-integration-gateway, and add your config.ini file and let it rip.
  4. Updated setup.py to reflect existing dependency with tlssyslog package. It's been deprecated and won't work with python 3.12+.

This PR introduces the ability to install falcon-integration-gateway
from the pypi repository and actually use it. A couple key changes were
needed for this to work:
1. Needed to update the setup.py file to include the relevant
directories that make the application work (ie fig/ config/)
2. Updated the config class to check the package level for the
defaults.ini file as the fig needs this to instantiate
3. Updated the config class to allow using local config files for
overrides. Now you can pip install falcon-integration-gateway, and add
your config.ini file and let it rip.
4. Updated setup.py to reflect existing dependency with tlssyslog
package. It's been deprecated and won't work with python 3.12+.
@carlosmmatos carlosmmatos added enhancement New feature or request bugfixes Fixes issues found in code labels Dec 19, 2024
@carlosmmatos carlosmmatos self-assigned this Dec 19, 2024
@carlosmmatos carlosmmatos merged commit f55d27e into CrowdStrike:main Dec 20, 2024
3 checks passed
@carlosmmatos carlosmmatos deleted the fix-pypi-package branch December 20, 2024 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfixes Fixes issues found in code enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants