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

Add support for Ubuntu 22.04 and Python 3.10+ #120

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

Conversation

dboyan
Copy link
Contributor

@dboyan dboyan commented Oct 26, 2022

Two changes are made:

  • Replace the qt5-default (no longer available for ubuntu 22.04) with the packages it refers to.
  • Default to PY_SSIZE_T_CLEAN, now mandatory with python 3.10. And fix the sizes from int to Py_ssize_t.

Boyan Ding added 2 commits October 26, 2022 09:27
Replace the meta package with the packages it refers to
In python 3.10, PY_SSIZE_T_CLEAN becomes mandatory and object size needs
to be represented with Py_ssize_t instead of int.
vinlin24 and others added 5 commits February 13, 2024 22:20
This file is a built binary, which shouldn't be tracked by version
control in the first place.
* .gitignore includes standard ignores for C, C++, and Python, pulled
  from https://github.com/github/gitignore.
* In addition, the .gitignore includes project-specific ignore rules,
  such as the wireshark directory generated from the installation
  process.
* Volatile files that shouldn't have been tracked in the first place,
  such as a .db and .txt file under examples/ have been untracked from
  version control. They will stay untracked now that ignore rules have
  been added for them.
When building Wireshark, the compiler throws an error for epan/epan.h.
The <glib.h> header, defined within the `extern "C"` block, has become
incompatible with pure C-style. To fix this, the `#include <glib.h>`
line had to be manually moved out of the `extern` block and recompiled.

Specs of the system on which the error was encountered:
* WSL: Ubuntu 22.04.2 LTS (GNU/Linux 5.15.133.1-microsoft-standard-WSL2
  x86_64)
* g++ (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
* gcc (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
* Wireshark version to install: 3.4.0

This warning should be removed if the change is fixed upstream (where
the Wireshark installation is sourced).
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