Skip to content

Commit 73dcf45

Browse files
author
stephan
committed
Minor code de-duplication across the top-level auto.def and autoconf/auto.def.
FossilOrigin-Name: 3c990d77cb83ab371d92c97e0528f790ab6126c60f48ff77e48bd166d7834cf0
1 parent 1392ff5 commit 73dcf45

File tree

5 files changed

+27
-19
lines changed

5 files changed

+27
-19
lines changed

Diff for: auto.def

-2
Original file line numberDiff line numberDiff line change
@@ -275,8 +275,6 @@ sqlite-handle-load-extension
275275
sqlite-handle-math
276276
sqlite-handle-icu
277277
sqlite-handle-emsdk
278-
sqlite-handle-common-feature-flags
279-
sqlite-show-feature-flags
280278
sqlite-process-dot-in-files
281279
sqlite-post-config-validation
282280
sqlite-dump-defines

Diff for: autoconf/auto.def

-2
Original file line numberDiff line numberDiff line change
@@ -90,8 +90,6 @@ sqlite-handle-line-editing
9090
sqlite-handle-load-extension
9191
sqlite-handle-math
9292
sqlite-handle-icu
93-
sqlite-handle-common-feature-flags
94-
sqlite-show-feature-flags
9593

9694
define ENABLE_LIB_SHARED [opt-bool shared]
9795
define ENABLE_LIB_STATIC [opt-bool static]

Diff for: autosetup/sqlite-config.tcl

+17-5
Original file line numberDiff line numberDiff line change
@@ -232,6 +232,10 @@ proc sqlite-setup-default-cflags {} {
232232
# from the legacy build and was missing the 3.48.0 release (the
233233
# initial Autosetup port).
234234
# https://sqlite.org/forum/forumpost/9801e54665afd728
235+
#
236+
# If any configure flags for features are in conflict with
237+
# CFLAGS-specified feature flags, all bets are off. There are no
238+
# guarantees about which one will take precedence.
235239
foreach cf [get-define CFLAGS ""] {
236240
switch -glob -- $cf {
237241
-DSQLITE_OMIT* -
@@ -307,8 +311,9 @@ proc sqlite-handle-common-feature-flags {} {
307311
}
308312
309313
#########################################################################
310-
# Show the final feature flag sets.
311-
proc sqlite-show-feature-flags {} {
314+
# Remove duplicates from the final feature flag sets and show them to
315+
# the user.
316+
proc sqlite-finalize-feature-flags {} {
312317
set oFF [get-define OPT_FEATURE_FLAGS]
313318
if {"" ne $oFF} {
314319
define OPT_FEATURE_FLAGS [lsort -unique $oFF]
@@ -319,7 +324,6 @@ proc sqlite-show-feature-flags {} {
319324
define OPT_SHELL [lsort -unique $oFF]
320325
msg-result "Shell options: [get-define OPT_SHELL]"
321326
}
322-
#parray ::sqliteConfig
323327
}
324328
325329
########################################################################
@@ -858,13 +862,21 @@ proc sqlite-handle-math {} {
858862
}
859863
860864
########################################################################
861-
# Generate the configure-process output file(s).
865+
# Perform some late-stage work and generate the configure-process
866+
# output file(s).
862867
proc sqlite-process-dot-in-files {} {
863868
########################################################################
864869
# When cross-compiling, we have to avoid using the -s flag to
865-
# /usr/bin/install: https://sqlite.org/forum/forumpost/9a67df63eda9925c
870+
# /usr/bin/install:
871+
# https://sqlite.org/forum/forumpost/9a67df63eda9925c
866872
define IS_CROSS_COMPILING $::sqliteConfig(is-cross-compiling)
867873
874+
# Finish up handling of the various feature flags here because it's
875+
# convenient for both the canonical build and autoconf bundles that
876+
# it be done here.
877+
sqlite-handle-common-feature-flags
878+
sqlite-finalize-feature-flags
879+
868880
########################################################################
869881
# "Re-export" the autoconf-conventional --XYZdir flags into something
870882
# which is more easily overridable from a make invocation. See the docs

Diff for: manifest

+9-9
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
C Disable\sa\stest\scase\sthat\s(intentionally)\sdoes\suse-after-free,\sas\sit\sdoes\noccasionally\scause\sproblems\seven\sfor\snon-sanitizer\sbuilds.
2-
D 2025-01-30T12:01:24.848
1+
C Minor\scode\sde-duplication\sacross\sthe\stop-level\sauto.def\sand\sautoconf/auto.def.
2+
D 2025-01-30T12:19:30.898
33
F .fossil-settings/empty-dirs dbb81e8fc0401ac46a1491ab34a7f2c7c0452f2f06b54ebb845d024ca8283ef1
44
F .fossil-settings/ignore-glob 35175cdfcf539b2318cb04a9901442804be81cd677d8b889fcc9149c21f239ea
55
F LICENSE.md e108e1e69ae8e8a59e93c455654b8ac9356a11720d3345df2a4743e9590fb20d
@@ -14,13 +14,13 @@ F art/sqlite370.eps aa97a671332b432a54e1d74ff5e8775be34200c2
1414
F art/sqlite370.ico af56c1d00fee7cd4753e8631ed60703ed0fc6e90
1515
F art/sqlite370.jpg d512473dae7e378a67e28ff96a34da7cb331def2
1616
F art/sqlite370.svg 40b7e2fe8aac3add5d56dd86ab8d427a4eca5bcb3fe4f8946cb3794e1821d531
17-
F auto.def caf7cca9c235fc6a3d7639e00eb75972d9bf8527717498b9fa1d7112d50b5720
17+
F auto.def e7e92090c98aeb0174d29988c259834eb1b71ae1ea927015c3ef300f6f9b68ae
1818
F autoconf/Makefile.fallback 22fe523eb36dfce31e0f6349f782eb084e86a5620b2b0b4f84a2d6133f53f5ac
1919
F autoconf/Makefile.in 56697ad25ecf23afa317148b06bdc14f85960b42e5ec434ac1ba87f63a3cb789
2020
F autoconf/Makefile.msc ffff61fe851443015ddb6600ab69a9df503cfec25459b336be7ba8c9a9e473f8
2121
F autoconf/README.first f1d3876e9a7852c22f275a6f06814e64934cecbc0b5b9617d64849094c1fd136
2222
F autoconf/README.txt 7f01dc3915e2d68f329011073662369e62a0938a2c69398807823c57591cb288
23-
F autoconf/auto.def 623e0d2339cbccc86b1c5e58d65702dc91a9fec4843a4c6198139e248579858e
23+
F autoconf/auto.def 23bc095a3890c0ca334abf7ef67d1c8af4c22c12832bcc738015e868d54fe9d7
2424
F autoconf/tea/Makefile.in ba0556fee8da09c066bad85a4457904e46ee2c2eabaa309c0e83a78f2f151a8e
2525
F autoconf/tea/README.txt 61e62e519579e4a112791354d6d440f8b51ea6db3b0bab58d59f29df42d2dfe3
2626
F autoconf/tea/aclocal.m4 52c47aac44ce0ddb1f918b6993e8beb8eee88f43
@@ -50,7 +50,7 @@ F autosetup/cc.tcl c0fcc50ca91deff8741e449ddad05bcd08268bc31177e613a6343bbd1fd3e
5050
F autosetup/jimsh0.c d40e381ea4526a067590e7b91bd4b2efa6d4980d286f908054c647b3df4aee14
5151
F autosetup/pkg-config.tcl 4e635bf39022ff65e0d5434339dd41503ea48fc53822c9c5bde88b02d3d952ba
5252
F autosetup/proj.tcl 9adf1539673cef15bff862d9360b479e6920cc2c0d85de707b0ba31c04ce4531
53-
F autosetup/sqlite-config.tcl e5de87d000ef5fa4ed78663c74688bbdd989b59bf5bae2aabaff65a502b0ec80
53+
F autosetup/sqlite-config.tcl 00af5b9d94d580367bf01984b86397e8d35b74090427def9591a54ded0e1a287
5454
F autosetup/system.tcl 51d4be76cd9a9074704b584e5c9cbba616202c8468cf9ba8a4f8294a7ab1dba9
5555
F configure 9a00b21dfd13757bbfb8d89b30660a89ec1f8f3a79402b8f9f9b6fc475c3303a x
5656
F contrib/sqlitecon.tcl 210a913ad63f9f991070821e599d600bd913e0ad
@@ -2209,8 +2209,8 @@ F tool/version-info.c 3b36468a90faf1bbd59c65fd0eb66522d9f941eedd364fabccd7227350
22092209
F tool/warnings-clang.sh bbf6a1e685e534c92ec2bfba5b1745f34fb6f0bc2a362850723a9ee87c1b31a7
22102210
F tool/warnings.sh 49a486c5069de041aedcbde4de178293e0463ae9918ecad7539eedf0ec77a139
22112211
F tool/win/sqlite.vsix deb315d026cc8400325c5863eef847784a219a2f
2212-
P ec71d9dcd582188d7fb4ab945187de1023eaaecffeeb2903a6fcc49650bd21e9
2213-
R 7b8d1f95b47e47a5d559070e34819714
2214-
U drh
2215-
Z 470d847ca31236034508764b46198386
2212+
P 23c7d5bea351c04785153ddb30e19c1236db9d11054ab8229893531026b53655
2213+
R ba9c9d617d2000ba71d01bbdea8f462f
2214+
U stephan
2215+
Z 94fb984dd769f6092f86cd16435effad
22162216
# Remove this line to create a well-formed Fossil manifest.

Diff for: manifest.uuid

+1-1
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
23c7d5bea351c04785153ddb30e19c1236db9d11054ab8229893531026b53655
1+
3c990d77cb83ab371d92c97e0528f790ab6126c60f48ff77e48bd166d7834cf0

0 commit comments

Comments
 (0)