lwprintf icon indicating copy to clipboard operation
lwprintf copied to clipboard

PlatformIO Registry

Open hagaigold opened this issue 3 years ago • 2 comments

Hi Tilen,

I am not sure this is an issue, but still...

Tried to consume the library thru PlatformIO IDE and had some issues building the library as-is. I have very little experience with PlatofrmIO but I assumed it should work out of the box.

Is the library.json file for use only by the PlatformIO? or have another purpose in life? After changing it a bit, I could build the library:

		]
	},
	"build": {
		"includeDir": "lwprintf/src/include",
		"srcDir": "lwprintf",
		"flags": [
			"-I../../include"
		]
    }
}

without the includeDir, I couldn't include the library without the srcDir, PlatoformIO try to build also the dev folder

the last bit, which I found helpful, is adding another include folder (-I) so I can place the options header, lwprintf_opts.h, in a project folder and not inside the library itself.

hagaigold avatar Jan 09 '22 15:01 hagaigold

I think you are right, problem is that there are also some files that are "optional" versus platform.

Could you prepare pull request with your changes?

MaJerle avatar Jan 13 '22 19:01 MaJerle

I have some concerns about the included path for the option header, lwprintf_opts.h: in the sample above I used relative path '-I../../include'. This path can be different if you use manual clone into the lib folder versus automatic install of the library using the lib_deps = majerle/LwPRINTF@^1.0.3 in platformio.ini.

one option is just to leave it for the user to set up a global include folder thru platformio.ini. another option is using an absolute path. e.g -I$PROJECT_INCLUDE_DIR, -I$PROJECT_DIR/<some folder>

hagaigold avatar Jan 13 '22 22:01 hagaigold