-
Notifications
You must be signed in to change notification settings - Fork 821
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
Expand tab to spaces and and cleanup formatting #4154
Conversation
@sauwming I had a hard time modifying aconfigure.ac during my previous PRs, so I decided to remove mixed indentation in aconfigure.ac first and clean up the formatting. Let's review this and I'll redo my PR on top. |
I think it should't format aconfigure which is generated by autoconf tool |
@sauwming is CI failing due to some enviromental issues? I'm not convinced, that formatting led to
|
@jimying unfortunately, autoconf 2.71 is not available in Debian 12 (current stable version of OS) |
@jimying sorry for inconvenience, it's Debian 11 here. We still need to build pjsp for that version of OS. If you insist, I'll regenerate with 2.71 |
This is a followup for pjsip#3292 and pjsip#3210 but regarding aconfigure.ac
1e41224
to
d874eaa
Compare
regenerated with autoconf 2.71 |
This is a followup for #3292 and #3210 but regarding aconfigure.ac