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.

Sign up for GitHub

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

Jump to bottom

modified: iptc/xtables.py #213

Merged
merged 2 commits into from
Sep 7, 2017
Merged

modified: iptc/xtables.py #213

merged 2 commits into from
Sep 7, 2017

Conversation

Pablo-Alvarez-Cordoba
Copy link
Contributor 

Add new xtdir to fix error on Debian

I'm using Debian 9.1 amd64, and if I do not run my script by:
XTABLES_LIBDIR=/usr/lib/x86_64-linux-gnu/xtables ./main.py
It does not work. I added this path to xtdir list and now it does works.

@palvarezcordoba
modified: iptc/xtables.py
b6a1c4c 
	Add new xtdir to fix error on Debian
@coveralls
Copy link

coveralls commented Aug 30, 2017 •
edited
Loading

Coverage Status

Coverage remained the same at 59.364% when pulling b6a1c4c on pajaroxdd:master into 2b78051 on ldx:master.

ldx
ldx requested changes Sep 1, 2017
View reviewed changes
Copy link
Owner

@ldx ldx left a comment

Choose a reason for hiding this comment

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

Sorry for nitpicking, but I prefer to keep lines <80 characters long. Can you update this to make sure that the line does not exceed 80 chars? Thanks!

@palvarezcordoba
modified: iptc/xtables.py
cd14daa 
	Strip lines to <80 chars long
@coveralls
Copy link

coveralls commented Sep 7, 2017 •
edited
Loading

Coverage Status

Coverage decreased (-0.1%) to 59.215% when pulling cd14daa on pajaroxdd:master into 2b78051 on ldx:master.

@Pablo-Alvarez-Cordoba
Copy link
Contributor Author

Now lines does not exceed 80 chars long.

@ldx ldx merged commit a425c01 into ldx:master Sep 7, 2017
@ldx
Copy link
Owner

ldx commented Sep 7, 2017

Thanks!

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

@ldx ldx ldx approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@Pablo-Alvarez-Cordoba @coveralls @ldx @palvarezcordoba