Macro DISALLOW_COPY_AND_ASSIGN include/file.h about shaka-packager

[joeyparrish]

I think you are correct. This is a violation that would break header installation for the shared library build. macros/classes.h should be moved to the public headers.

[zaki699]

Yes that’s a bit of a problem on my side as I am currently using libpackager. I have to manually copy the file macros/classes into include which is not great. Should I create a pull request to add macros/classes to the public include ?

[joeyparrish]

Yes, please!

[joeyparrish]

There is a CMake target that is meant to catch errors like these. I'm going to figure out why it didn't.

[joeyparrish]

Commands for configuring shared libs and building the link test, which compiles a simple application with libpackager and the public headers:

cmake -S . -B build/ -DCMAKE_BUILD_TYPE=Debug -G Ninja -DBUILD_SHARED_LIBS=ON
cmake --build build/ --parallel -- packager_link_test

It works. However, the main file includes only packager.h, which includes all the other header files except file.h. If we add file.h to packager.h, the build fails.