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

ntopng: update to v6.2 #6220

Merged
merged 3 commits into from
Oct 31, 2024
Merged

ntopng: update to v6.2 #6220

merged 3 commits into from
Oct 31, 2024

Conversation

hgy59
Copy link
Contributor

@hgy59 hgy59 commented Sep 3, 2024

Description

  • update ntopng and dependencies
  • update libpcap from v1.10.4 to v1.10.5
  • update ndpi from v3.4 to v4.10
  • update libcap from v2.66 to v2.70
  • update json-c from v0.16-20220414 to v0.17-20230812
  • add hiredis v1.2.0
  • add native/nodejs_16 required to build the web ui

Fixes #

Checklist

  • Build rule all-supported completed successfully
  • New installation of package completed successfully
  • Package upgrade completed successfully (Manually install the package again)
  • Package functionality was tested
  • Any needed documentation is updated/created

Type of change

  • Package update

@hgy59
Copy link
Contributor Author

hgy59 commented Sep 3, 2024

This PR superseeds #6216

@hgy59
Copy link
Contributor Author

hgy59 commented Sep 16, 2024

@th0ma7 currently the build of ntopng fails for native libmysqlclient when building mysqlconnector-c. Both fail, the former ntopng on the master branch (seen with merged #6231 https://github.com/SynoCommunity/spksrc/actions/runs/10875221744) and this PR with updated ntopng after merging in the current master.
The former build succeeded (only homeasistant faild, see https://github.com/SynoCommunity/spksrc/actions/runs/10690439769?pr=6220)

The build fails with

make[6]: Entering directory '/github/workspace/native/libmysqlclient/work-native/mysql-connector-c-6.1.11-src/build'
make[6]: *** No rule to make target 'install'.  Stop.

First I thought it was broken by #6212, but #6212 was already merged before.
The next candidate is #6198
I guess it must be related to a cmake/ninja mk file (missing target install)

Unfortunately I am offline for a few weeks and can't dig into it...
Can you have a look?

@th0ma7
Copy link
Contributor

th0ma7 commented Sep 16, 2024

10-4, Will do.

@th0ma7
Copy link
Contributor

th0ma7 commented Sep 22, 2024

@hgy59 now fixed with #6235

@hgy59 hgy59 reopened this Sep 24, 2024
- update ntopng and dependencies
- update libpcap from v1.10.4 to v1.10.5
- update ndpi from v3.4 to v4.10
- update libcap from v2.66 to v2.70
- update json-c from v0.16-20220414 to v0.17-20230812
- add hiredis v1.2.0
- add native/nodejs_16 required to build the web ui
@hgy59 hgy59 merged commit e487345 into SynoCommunity:master Oct 31, 2024
15 checks passed
@hgy59 hgy59 deleted the update_ntopng branch October 31, 2024 19:08
@hgy59 hgy59 added the status/published Published and activated (may take up to 48h until visible in DSM package manager) label Nov 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/published Published and activated (may take up to 48h until visible in DSM package manager)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants