Skip to content

Commit f5ee936

Browse files
andyclaude
andcommitted
CMakeLists: strip 3 corrupt filenames from vendored eccehelp.tar during extraction
CI's RPM packaging job has been silently failing (CPack Error: Problem compressing the directory / rpmbuild: File not found) since the cpack packaging step was added -- root cause confirmed via `tar tf` directly against the vendored tarball: 3 entries under basis/ and builder/ have literal HTML markup embedded in their filenames instead of real names (e.g. `overview_top"> <!-- #begineditable "title.html`), decades-old corruption in the vendored PNNL content itself. Harmless for a DEB build (dpkg-deb tolerates it) but real disqualifying breakage for RPM's stricter file-list handling -- and not something to rely on being safe under every filesystem/locale a DEB build might run on either. None of the 3 are reachable from any real help.urls navigation path (their "content" is a stray HTML comment fragment, not a page), so deleting them during extraction costs nothing. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
1 parent 99ef0ce commit f5ee936

1 file changed

Lines changed: 15 additions & 0 deletions

File tree

‎CMakeLists.txt‎

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -474,6 +474,21 @@ if(EXISTS ${_ecce_help_tarball})
474474
COMMAND ${CMAKE_COMMAND} -E make_directory ${CMAKE_BINARY_DIR}/ecce-help-extracted/EcceHelp
475475
COMMAND ${CMAKE_COMMAND} -E chdir ${CMAKE_BINARY_DIR}/ecce-help-extracted/EcceHelp
476476
${CMAKE_COMMAND} -E tar xf ${_ecce_help_tarball}
477+
# The vendored tarball itself ships 3 genuinely corrupt entries under
478+
# basis/ and builder/ -- filenames containing literal HTML markup
479+
# (quotes, "<", ">", spaces, e.g. `overview_top"> <!-- #begineditable
480+
# "title.html`), not real help pages. Confirmed via `tar tf` against
481+
# the tarball directly -- this is decades-old corruption in the
482+
# vendored PNNL content itself, not something this build introduced.
483+
# Harmless for a DEB build (dpkg-deb tolerates the odd filename) but
484+
# a real, reproducible CI failure for RPM packaging specifically
485+
# ("CPack Error: Problem compressing the directory" -- rpmbuild's
486+
# file-list handling rejects them outright), and not guaranteed safe
487+
# on every filesystem/locale a DEB build might run under either.
488+
# None of these 3 files are reachable from help.urls or any real
489+
# navigation path (their "content" is a stray HTML comment fragment,
490+
# not a page), so deleting them costs nothing.
491+
COMMAND bash -c "find '${CMAKE_BINARY_DIR}/ecce-help-extracted/EcceHelp' -name '*begineditable*' -delete"
477492
WORKING_DIRECTORY ${CMAKE_BINARY_DIR}
478493
COMMENT "Extracting vendored ECCE help content"
479494
)

0 commit comments

Comments
 (0)