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

Fixes #12

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Fixes #12

wants to merge 17 commits into from

Conversation

cassella
Copy link
Collaborator

@cassella cassella commented Sep 1, 2015

Here are a few more small commits to fix small compiler warnings.

Plus one I'd neglected to submit a pull-request for. In February. Of 2011.

I've been getting this error from configure:

  checking for id3lib version... ./configure: line 11287:
  .: conftest.id3: file not found

strace found it was not looking in ".".  The bash man page says about
the source command:

  If filename does not contain a slash, file names in PATH are used to
  find the directory containing file- name.

  When bash is not in posix mode, the current directory is
  searched if no file is found in PATH.

Presumably this is what's happening.  So include ./ in the filename to source.
gcc issues a warning about the latter.
Seems crazy, but that's what the internet suggests.
They are not modified, and this addresses a compiler warning.
(The case shouldn't happen anyway.)
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.

1 participant