Skip to content

Conversation

@yadij
Copy link
Contributor

@yadij yadij commented Jan 7, 2026

Use 'dnl' instead of hash to elide the unnecessary blurbs
from generated output.

Use 'dnl' instead of hash to elide the unnecessary
blurbs from generated output.
@yadij
Copy link
Contributor Author

yadij commented Jan 7, 2026

Example of the change in ./configure script:

@@ -5372,12 +5342,6 @@
 
 
 
-## Copyright (C) 1996-2026 The Squid Software Foundation and contributors
-##
-## Squid software is distributed under GPLv2+ license and includes
-## contributions from numerous individuals and organizations.
-## Please see the COPYING and CONTRIBUTORS files for details.
-##
 
 
 

There is no logic, behaviour or legal change. Though CREDITS file has been updated to show the new prefix characters on each line.

# https://www.gnu.org/software/autoconf-archive/ax_cxx_compile_stdcxx.html
# ===========================================================================
#
# SYNOPSIS
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not think it is a good idea to change copied/imported code this way because it complicates spotting differences with official macro versions, but that problem is so minor that I am not going to block this PR.

@yadij yadij added M-cleared-for-merge https://github.com/measurement-factory/anubis#pull-request-labels S-could-use-an-approval An approval may speed this PR merger (but is not required) labels Jan 8, 2026
squid-anubis pushed a commit that referenced this pull request Jan 17, 2026
Use 'dnl' instead of hash to elide the unnecessary blurbs
from generated output.
@squid-anubis squid-anubis added the M-waiting-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels label Jan 17, 2026
@squid-anubis squid-anubis added M-merged https://github.com/measurement-factory/anubis#pull-request-labels and removed M-waiting-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels M-cleared-for-merge https://github.com/measurement-factory/anubis#pull-request-labels labels Jan 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

M-merged https://github.com/measurement-factory/anubis#pull-request-labels S-could-use-an-approval An approval may speed this PR merger (but is not required)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants