Mercurial > vim
annotate ci/unlisted.make @ 33988:7c30841c60a0 v9.0.2180
patch 9.0.2180: POSIX function name in exarg causes issues
Commit: https://github.com/vim/vim/commit/6fdb6280821a822768df5689a5d727e37d38306c
Author: Zoltan Arpadffy <zoltan.arpadffy@gmail.com>
Date: Tue Dec 19 20:53:07 2023 +0100
patch 9.0.2180: POSIX function name in exarg causes issues
Problem: POSIX function name in exarg struct causes issues
on OpenVMS
Solution: Rename getline member in exarg struct to ea_getline,
remove isinf() workaround for VMS
There are compilers that do not treat well POSIX functions - like
getline - usage in the structs.
Older VMS compilers could digest this... but the newer OpenVMS compilers
( like VSI C x86-64 X7.4-843 (GEM 50XB9) ) cannot deal with these
structs. This could be limited to getline() that is defined via
getdelim() and might not affect all POSIX functions in general - but
avoiding POSIX function names usage in the structs is a "safe side"
practice without compromising the functionality or the code readability.
The previous OpenVMS X86 port used a workaround limiting the compiler
capabilities using __CRTL_VER_OVERRIDE=80400000
In order to make the OpenVMS port future proof, this pull request
proposes a possible solution.
closes: #13704
Signed-off-by: Zoltan Arpadffy <zoltan.arpadffy@gmail.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>
author | Christian Brabandt <cb@256bit.org> |
---|---|
date | Tue, 19 Dec 2023 21:00:04 +0100 |
parents | e6c291944e18 |
children |
rev | line source |
---|---|
33861
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
1 # vim: ft=make |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
2 SHELL = /bin/bash |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
3 |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
4 # List all files that are tracked in git but not listed in Filelist. |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
5 # Exit code is 2 ("Make encountered an error") if any such files exist. |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
6 |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
7 # Filelist is a Makefile that defines many variables, so we use Make itself to |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
8 # query which variables it defines, then expand them all by wrapping each name |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
9 # in $(...), importing Filelist and using $(eval). |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
10 |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
11 include Filelist |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
12 $(eval all_patterns := $(shell \ |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
13 make -f Filelist --question --print-data-base --no-builtin-rules \ |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
14 --no-builtin-variables 2>/dev/null \ |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
15 | sed -nre \ |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
16 '/^# makefile .from \x27Filelist\x27,/ { \ |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
17 n; \ |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
18 s/ = .*//; \ |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
19 T; \ |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
20 s/.*/$$(\0)/; \ |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
21 p; \ |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
22 }')) |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
23 |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
24 # In Makefile's `prepeare` target, all the IN_README_DIR files are moved from |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
25 # READMEdir to the root, so add those files in their Git-tracked location: |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
26 all_patterns := $(all_patterns) \ |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
27 $(foreach readme, $(IN_README_DIR), READMEdir/$(readme)) |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
28 |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
29 # The result 'all_patterns' is a list of patterns (globs), which we expand with |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
30 # wildcard to get actual filenames. Note this means Filelist can list a file |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
31 # that does not exist, and it will be omitted at this step. |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
32 listed_files := $(wildcard $(all_patterns)) |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
33 |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
34 # Default target to actually run the comparison: |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
35 .PHONY: check |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
36 check: |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
37 @# There are too many files to list on the command line, so we write |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
38 @# that to a temporary file, one per line. |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
39 $(file > Filelist-listed-files) |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
40 $(foreach filename, $(listed_files),\ |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
41 $(file >> Filelist-listed-files,$(filename))) |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
42 @# Compare the sorted lists. Delete that temporary file on both |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
43 @# success and failure, but exit with diff's exit code. |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
44 diff -u0 --label files-in-git <(git ls-files | sort) \ |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
45 --label Filelist <(sort --unique Filelist-listed-files); \ |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
46 RV=$$?; \ |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
47 rm Filelist-listed-files; \ |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
48 (($$RV != 0)) && echo "Add files to the right variable in Filelist."; \ |
e6c291944e18
CI: check that all files are listed in Filelist (#13601)
Christian Brabandt <cb@256bit.org>
parents:
diff
changeset
|
49 exit $$RV |