christgau / wsdd

A Web Service Discovery host daemon.
MIT License
814 stars 98 forks source link

Invalid syntax #120

Closed kmsmith222 closed 2 years ago

kmsmith222 commented 2 years ago

OS : Ubuntu 20.04 I did change the interfaces names to ethX, not sure if that has anything to do with it.

When i restart WSDD this shows up in the log: -- wsdd[18338]: File "/usr/bin/wsdd", line 47 wsdd[18338]: class MulticastHandler: wsdd[18338]: ^ wsdd[18338]: SyntaxError: invalid syntax

And it fails to start..Tisis new since I havent had a reason to mess with it for a few months.

christgau commented 2 years ago

The source code in the repo didn't change for a while. So if you already had 0.6.4 running, I'd suggest that wsdd.py has been modified locally on your system, because Syntax Error at that position is something that very likely would have been seen during development and by other users as well.

If you used the packages for installation, consider reinstalling them. Otherwise download the latest release and install that one. You could also check that the Python interpreter is correct.

kmsmith222 commented 2 years ago

I know they weren't altered here as I didn't mess with it since installation. Is python picky about trailing spaces? Maybe a python update broke it, I'll look into it closer. Thanks. -------- Original message --------From: Steffen Christgau @.> Date: 10/25/21 13:03 (GMT-05:00) To: christgau/wsdd @.> Cc: kmsmith222 @.>, Author @.> Subject: Re: [christgau/wsdd] Invalid syntax (Issue #120) The source code in the repo didn't change for a while. So if you already had 0.6.4 running, I'd suggest that wsdd.py has been modified locally on your system, because Syntax Error at that position is something that very likely would have been seen during development and by other users as well. If you used the packages for installation, consider reinstalling them. Otherwise download the latest release and install that one. You could also check that the Python interpreter is correct.

christgau commented 2 years ago

This really does not appear to be an issue on the wsdd side. There is also an action since a few commits that checks for syntax errors: https://github.com/christgau/wsdd/blob/b86c24190d8ea169db68eb0658f23ca44dadb095/.github/workflows/static-analyses.yml#L23-L24

and it did not reveal any. Therefore I am closing this issue.