Skip to content

Commandline options, second try #37

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

Merged
merged 6 commits into from
Aug 7, 2024
Merged

Conversation

zabbius
Copy link
Contributor

@zabbius zabbius commented Aug 7, 2024

Hi, ported my PR to new main branch. Has one question. How to use config variable from youtubeUnblock.c in mangle.c?

@Waujito
Copy link
Owner

Waujito commented Aug 7, 2024

Hi! Thank you!

I think if you just declare this variable as non-static in config.h and define in youtubeUnblock .c everything will be fine.

@Waujito Waujito marked this pull request as ready for review August 7, 2024 05:42
@jelly8173
Copy link

Why not use something like getopt? Just curious

@zabbius
Copy link
Contributor Author

zabbius commented Aug 7, 2024

Why not use something like getopt? Just curious

Because I did not used C for many many years. Just forgot everything =)
You're right, if getopt is available as simple as printf we should use it.

@jelly8173
Copy link

if getopt is available as simple as printf we should use it.

Its part of libc, ubiquitous in a linux world and one of the standard options to choose from when it comes to parse some args

@Waujito Waujito merged commit eac78de into Waujito:kern_mod Aug 7, 2024
Waujito added a commit that referenced this pull request Aug 13, 2024
Check #56 and #37 for more details. Real commit made to openwrt branch. Pushing this to restart actions
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.

3 participants