Bug 26673 removed the shebang from SIPServer.pm however despite its name it is not a module but a program if started other than 'perl SIPServer.pm' it will fail to run as no interpreter is found. This may effect custom startup files or attempts to test/debug by developers.
Why the misleading name? dont know, I think it reflects the heritage of OpenSIP which tried to do some clever things without fully understanding them leaving the rest of us to puzzle over the code subsequently
Created attachment 118994 [details] [review]
patch restoring the shebang
restore the shebang so that
is again equivalent of
perl SIPServer.pm config_file
Doh! This ought to have been caught at QA for bug 26673, but knowledge around SIP is somewhat limited in the QA circle :(.
Going for a straight PQA on this one.. it's a trivial and obvious correction to that initial mistake.
Created attachment 118996 [details] [review]
Bug 28054 Reinstate the shebang in SIPServer.pm
Bug 26673 renoved the shebang from SIPServer.pm. This was an error
SIPServer.pm is a program not a perl module, and needs the shebang
for the OS to assign an interpreter unless the command line
calls it as an argument to perl. Restore the shebang
Signed-off-by: Martin Renvoize <email@example.com>
Pushed to master for 21.05, thanks to everybody involved!
Yikes... looking at SIPServer.pm and shaking my head.
If we cared enough, we'd move that loose code in SIPServer.pm into a function, and then create a sip_server.pl script that invokes it.
I discussed that with Colin just prior the his retiring. He suggested it wasn't as simple as it looked.
I'd love to clean up the sip code at some point, but alas I don't know if/when I'll have time to spend on that :(
Pushed to 20.11.x for 20.11.05
(In reply to Martin Renvoize from comment #6)
> I discussed that with Colin just prior the his retiring. He suggested it
> wasn't as simple as it looked.
I didn't realize that Colin had retired! When was that?
> I'd love to clean up the sip code at some point, but alas I don't know
> if/when I'll have time to spend on that :(
Hard relate. I only have maybe 5 out of 100 libraries using SIP so it's not really a work-related priority.
Pushed to 20.05.x for 20.05.11
Not backported to oldoldstable (19.11.x). Feel free to ask if it's needed.