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

Fixed a bug that does not work properly if the project path contains … #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kazuhidet
Copy link

@kazuhidet kazuhidet commented May 25, 2018

…spaces.

If the path of the project to be analyzed by OCLint includes spaces as shown in the following example, an error occurs in the existence confirmation of the target path and it is not processed normally.

Project path like:
/Users/me/My Project/MyApp/Someware

For example, in compile_commands.json created with the option -r json-compilation-database in the xcpretty command, the space contained in the path is escaped with a backslash like this, but oclint-json-compilation-database do not care about this.

JSON contained like this:
“/Users/me/My\ Project/MyApp/Someware”

Likewise, in the path of the compile command returned by clang::tooling::CompileCommand in oclint-driver/lib/Driver.cpp, escape with backslash well if the path contains spaces. It also needed a fix.

@brum-cit
Copy link

brum-cit commented Apr 15, 2024

Hey Guys, I'm facing a problem that seems similar to this. I'm dealing with a project that has some files with special character "~" in the name. E.g.:

iPad/Coupons/CouponCell~iPad.m

And I get this error when trying to run oclint without excluding it:
oclint: error: Cannot change dictionary into "/Users/<username>/Documents/Projects/<prj_name>/iPad/Coupons/CouponCell~iPad.m", please make sure the directory exists and you have permission to access!

PS: I'm sorry but I couldnt find anywhere else to create a ticket and discuss this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants