Add more documentation on building the tools

[kingsawyer]

The instructions for building the writer app don't include the filename. Adding this makes the following steps which assume the file is named "writer.cpp" work seamlessly.

There are no instructions on building the tools and unless you've used cmake you would be stuck wondering.

[linux-foundation-easycla]

• ❌ - login: @kingsawyer / name: Dave Sawyer . The commit (9ac21f1, 5406064, 68fee9d, 656cb35, ab05620) is not authorized under a signed CLA. Please click here to be authorized. For further assistance with EasyCLA, please submit a support request ticket.

[cary-ilm]

Thank you for the contribution, we appreciate anything that helps make things more clear.

The Hello, World example reader and writer code are meant to be examples of using OpenEXR as a library, separate from building the library and tools itself, so these instructions should address the needs of someone on a system with the libraries installed, without access to the actual library source code. That's a good suggestion to make the code downloadable, I admit that didn't occur to me. Rather than simply giving the filename as writer.cpp, could you make that rst reference a link to actually download the file? The same would apply to the example CMakeLists.txt and reader.cpp.

[cary-ilm]

This page isn't intended to discuss building the tools, just how to use them, so I don't think this is appropriate here.

[cary-ilm]

I also think this is where the error is that causes the build to fail, see the "Website/Website" link below.

[kingsawyer]

Where would I learn how to build the tools? Should we add a readme.md to the tools directory?

[cary-ilm]

The tools get built as a component of the full project build as long as OPENEXR_BUILD_TOOLS is ON, which is is by default. But I guess that's not entirely obvious, so worth mentioning in the other parts of the documentation.

[cary-ilm]

But the instructions below won't work because the tool directory is not set up for an independent build. The more appropriate thing to say here is something like the previous comment, the tools are built as a component of the overall project build when OPENEXR_BUILD_TOOLS is ON.

And the indentation is off, so it's causing the website build to fail as well.

[cary-ilm]

Apologies for letting this sit, but we would indeed like to accept your submission. The ASWF project policy is to require a signed contributor license agreement. Can you click on the red "NOT COVERED" link above and fill out the form? Thanks.

[cary-ilm]

The confusion cited here and the suggestions for explicitly-named downloadable files are hopefully address by #1620 and #1622