it looks the 4.1.1 RPM no longer includes a post-i...
# core
it looks the 4.1.1 RPM no longer includes a post-install script; is this intentional?
4.0.2 did, fwiw
if i had to guess i'd say 4.0.2 was built w/ the FPM based stuff (tools/deployment/ which does include the --post-install flag, but it looks like packaging.cmake needs something along the lines of set(CPACK_RPM_POST_INSTALL_SCRIPT_FILE "${CMAKE_SOURCE_DIR}/tools/deployment/")
good catch
i'll definitely test this out. i didn't check debs yet, thanks for that! BTW, i think you want CPACK_RPM_PACKAGE_RELEASE instead of CPACK_RPM_PACKAGE_RELEASE_DIST? i think cpack is actually ignoring this, because if you bump OSQUERY_RELEASE_VERSION the RPM will keep the same release; changing CPACK_RPM_PACKAGE_RELEASE_DIST to CPACK_RPM_PACKAGE_RELEASE fixes it
so it looks like this works for RPMs! but it looks like debs are already fine (i am not a deb expert by any means, but
Copy code
dpkg -e osquery_4.1.1_1.linux.amd64.deb
seems to show the postinst). i think the lines you're adding on 180 and 181 are already present on 128 and 129:
tbq i don't know why it works without
Copy code
but i assume cpack is figuring that out on its own currently?