mobile-insight / mobileinsight-core

Mobile Network Intelligence Made Easy -- MobileInsight core functionalities
http://www.mobileinsight.net
Other
106 stars 82 forks source link

Apply ignore rules #137

Closed vinlin24 closed 8 months ago

vinlin24 commented 8 months ago

Untrack Volatile Files

Firstly, some files that should not have been tracked by version control in the first place have been removed:

Testing with examples/offline-analysis-example.py confirms that it still works even after removing these 3 files (the latter two are regenerated as part of its runtime anyway).

Implement Ignore Rules

To enforce version control best practices, this PR introduces a repository .gitignore file. I populated it with standard ignore rules for C, C++, and Python, pulled from GitHub's official .gitignore templates.

In addition, I appended project-specific ignore rules. These are documented in the .gitignore file as comments.

Warning For Wireshark Build Error

Finally, not directly related to ignore rules but important, recent builds have encountered an error when building Wireshark due to its epan/epan.h including the <glib.h> header under extern "C". We presume glib.h is no longer compatible with C linkage. To fix this, the #include <glib.h> line had to be manually moved out of the extern block and recompiled.

Since this is a change in upstream source code, it cannot be fixed in version control for this repository. As a temporary workaround, a warning has been added to the README under installation instructions describing this error and how to fix it. This warning should be removed if the change is fixed upstream (where the Wireshark installation is sourced).