Skip to content

Commit

Permalink
install-qa-check.d/10ignored-flags: Update two comments
Browse files Browse the repository at this point in the history
Signed-off-by: Ulrich Müller <[email protected]>
  • Loading branch information
ulm committed Jan 6, 2025
1 parent 6417df1 commit c52e71b
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions bin/install-qa-check.d/10ignored-flags
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,8 @@ ignored_flag_check() {
# Check for files built without respecting *FLAGS. Note that
# -frecord-gcc-switches must be in all *FLAGS variables, in
# order to avoid false positive results here.
# NOTE: This check must execute before prepall/prepstrip, since
# prepstrip strips the .GCC.command.line sections.
# NOTE: This check must execute before estrip --prepallstrip,
# since estrip strips the .GCC.command.line sections.
if [[ "${CFLAGS}" == *-frecord-gcc-switches* ]] && \
[[ "${CXXFLAGS}" == *-frecord-gcc-switches* ]] && \
[[ "${FFLAGS}" == *-frecord-gcc-switches* ]] && \
Expand All @@ -30,7 +30,7 @@ ignored_flag_check() {
for x in $(scanelf -qyRF '#k%p' -k '!.GCC.command.line' "${ED%/}/") ; do
# Separate out file types that are known to support
# .GCC.command.line sections, using the `file` command
# similar to how prepstrip uses it.
# similar to how estrip uses it.
f=$(file -S "${x}") || continue
[[ -z ${f} ]] && continue
if [[ ${f} == *"SB executable"* || ${f} == *"SB pie executable"* ||
Expand Down

0 comments on commit c52e71b

Please sign in to comment.