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

fixes to the debian packaging #62

Merged
merged 9 commits into from Sep 18, 2012
Merged

fixes to the debian packaging #62

merged 9 commits into from Sep 18, 2012

Conversation

obilodeau
Copy link
Contributor

@fdurand please review commit per commit and once you are ok with the changes, we'll send this into integration.

bug covered:

Unfortunately current integration is also still broken because of fix/pfcmd-suid so we will have to fix that first.

NEWS

Enhancements
 * Debian packages improvements

Bug fixes
 * Debian: issues with translations (#1465)
 * Debian: SSH and Telnet based deauthentication (#1511)

- Just like RHEL
- Why wasting client-side CPU cycles?
- there were a LOT of duplicated entries
- also tried to quickly organize / document things but it's not complete yet
It's user controlled. If a user would have enabled radius auth he would
have seen failures because the module is not installed.
since we already depend on packetfence
Will avoid problems if debian ever ships a more recent version and we haven't
yet updated to the new API.
@ghost ghost assigned fdurand Sep 12, 2012
# perl stuff
libapache-htpasswd-perl, libbit-vector-perl, libtext-csv-perl,
# perl basic components
perl-suid, liblist-moreutils-perl, libwww-perl, libtry-tiny-perl
Copy link
Member

Choose a reason for hiding this comment

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

Missing comma

Copy link
Contributor Author

Choose a reason for hiding this comment

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

resolved

@obilodeau
Copy link
Contributor Author

Should fix #1511: SSH and Telnet deAuth doesn't work on debian too now. Updated NEWS entry in pull request to reflect that.

Ready for another round of review please.

@fdurand
Copy link
Member

fdurand commented Sep 18, 2012

ok to merge

obilodeau added a commit that referenced this pull request Sep 18, 2012
@obilodeau obilodeau merged commit f22ccf1 into stable Sep 18, 2012
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