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

Incorrect fsf address in header files #56

Open
sanjayankur31 opened this issue May 26, 2020 · 2 comments
Open

Incorrect fsf address in header files #56

sanjayankur31 opened this issue May 26, 2020 · 2 comments
Assignees

Comments

@sanjayankur31
Copy link

Hi there,

Fedora maintainer for lingot here. I was just updating to the latest release 1.1.1, and rpmlint pointed out that the FSF address in the headers is wrong:

lingot-devel.x86_64: E: incorrect-fsf-address /usr/include/lingot/lingot-audio-alsa.h
lingot-devel.x86_64: E: incorrect-fsf-address /usr/include/lingot/lingot-audio-jack.h
lingot-devel.x86_64: E: incorrect-fsf-address /usr/include/lingot/lingot-audio-oss.h
lingot-devel.x86_64: E: incorrect-fsf-address /usr/include/lingot/lingot-audio-pulseaudio.h
lingot-devel.x86_64: E: incorrect-fsf-address /usr/include/lingot/lingot-audio.h
lingot-devel.x86_64: E: incorrect-fsf-address /usr/include/lingot/lingot-config-scale.h
lingot-devel.x86_64: E: incorrect-fsf-address /usr/include/lingot/lingot-config.h
lingot-devel.x86_64: E: incorrect-fsf-address /usr/include/lingot/lingot-core.h
lingot-devel.x86_64: E: incorrect-fsf-address /usr/include/lingot/lingot-defs.h
lingot-devel.x86_64: E: incorrect-fsf-address /usr/include/lingot/lingot-filter.h
lingot-devel.x86_64: E: incorrect-fsf-address /usr/include/lingot/lingot-msg.h

Could you please take a look when you have a minute and update it?

@ibancg ibancg self-assigned this May 26, 2020
@ibancg
Copy link
Owner

ibancg commented May 26, 2020

Hi Ankur, thanks for pointing it out.

It should be fixed after #57, already merged into master. I hope the wording is correct, as I don't have those lint checks to test it myself.

I think I will roll it out as release 1.1.2.

@sanjayankur31
Copy link
Author

Thanks for the quick reply @ibancg . Looks good, but I'll try to find the time to make a temporary package from the current snapshot and run the linter on it this week.

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

No branches or pull requests

2 participants