open.c: At top level:
open.c:276:1: error: return type defaults to 'int' [-Wimplicit-int]
276 | fk_open(int seq, int fmt, ftnint n)
| ^~~~~~~
open.c: In function 'fk_open':
open.c:286:15: warning: assignment discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
286 | a.oacc= seq==SEQ?"s":"d";
| ^
open.c:287:15: warning: assignment discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
287 | a.ofm = fmt==FMT?"f":"u";
| ^
open.c: In function 'f_open':
open.c:232:20: warning: this statement may fall through [-Wimplicit-fallthrough=]
232 | if (!access(buf,0))
| ^
open.c:236:9: note: here
236 | case 'r': /* Fortran 90 replace option */
| ^~~~
make[7]: *** [Makefile:677: open.lo] Error 1
make[7]: Leaving directory '/tmp/BUILD.fc40.x86_64/trinity-digikam-14.1.2~pre17+8f0b4a3a/digikam/imageplugins/coreplugin/sharpnesseditor/clapack'
make[6]: *** [Makefile:699: all-recursive] Error 1
make[6]: Leaving directory '/tmp/BUILD.fc40.x86_64/trinity-digikam-14.1.2~pre17+8f0b4a3a/digikam/imageplugins/coreplugin/sharpnesseditor'
make[5]: *** [Makefile:837: all-recursive] Error 1
make[5]: Leaving directory '/tmp/BUILD.fc40.x86_64/trinity-digikam-14.1.2~pre17+8f0b4a3a/digikam/imageplugins/coreplugin'
make[4]: *** [Makefile:562: all-recursive] Error 1
make[4]: Leaving directory '/tmp/BUILD.fc40.x86_64/trinity-digikam-14.1.2~pre17+8f0b4a3a/digikam/imageplugins'
make[3]: *** [Makefile:615: all-recursive] Error 1
make[3]: Leaving directory '/tmp/BUILD.fc40.x86_64/trinity-digikam-14.1.2~pre17+8f0b4a3a/digikam'
make[2]: *** [Makefile:615: all-recursive] Error 1
make[2]: Leaving directory '/tmp/BUILD.fc40.x86_64/trinity-digikam-14.1.2~pre17+8f0b4a3a'
make[1]: *** [Makefile:532: all] Error 2
make[1]: Leaving directory '/tmp/BUILD.fc40.x86_64/trinity-digikam-14.1.2~pre17+8f0b4a3a'
<!--
This is a comment.
Please fill in the required fields below.
The comments provide instructions on how to do so.
Note: You do not need to remove comments.
-->
## Basic information
- TDE version: 14.1.2
- Distribution: Fedora 40
- Hardware: amd64
<!--
Use SL/* labels to set the severity level.
Please do not set a milestone.
-->
## Description
FTBFS
## Steps to reproduce
Build
## Screenshots
<!-- If it seems useful, please provide provide one or more screenshots. -->
```
open.c: At top level:
open.c:276:1: error: return type defaults to 'int' [-Wimplicit-int]
276 | fk_open(int seq, int fmt, ftnint n)
| ^~~~~~~
open.c: In function 'fk_open':
open.c:286:15: warning: assignment discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
286 | a.oacc= seq==SEQ?"s":"d";
| ^
open.c:287:15: warning: assignment discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
287 | a.ofm = fmt==FMT?"f":"u";
| ^
open.c: In function 'f_open':
open.c:232:20: warning: this statement may fall through [-Wimplicit-fallthrough=]
232 | if (!access(buf,0))
| ^
open.c:236:9: note: here
236 | case 'r': /* Fortran 90 replace option */
| ^~~~
make[7]: *** [Makefile:677: open.lo] Error 1
make[7]: Leaving directory '/tmp/BUILD.fc40.x86_64/trinity-digikam-14.1.2~pre17+8f0b4a3a/digikam/imageplugins/coreplugin/sharpnesseditor/clapack'
make[6]: *** [Makefile:699: all-recursive] Error 1
make[6]: Leaving directory '/tmp/BUILD.fc40.x86_64/trinity-digikam-14.1.2~pre17+8f0b4a3a/digikam/imageplugins/coreplugin/sharpnesseditor'
make[5]: *** [Makefile:837: all-recursive] Error 1
make[5]: Leaving directory '/tmp/BUILD.fc40.x86_64/trinity-digikam-14.1.2~pre17+8f0b4a3a/digikam/imageplugins/coreplugin'
make[4]: *** [Makefile:562: all-recursive] Error 1
make[4]: Leaving directory '/tmp/BUILD.fc40.x86_64/trinity-digikam-14.1.2~pre17+8f0b4a3a/digikam/imageplugins'
make[3]: *** [Makefile:615: all-recursive] Error 1
make[3]: Leaving directory '/tmp/BUILD.fc40.x86_64/trinity-digikam-14.1.2~pre17+8f0b4a3a/digikam'
make[2]: *** [Makefile:615: all-recursive] Error 1
make[2]: Leaving directory '/tmp/BUILD.fc40.x86_64/trinity-digikam-14.1.2~pre17+8f0b4a3a'
make[1]: *** [Makefile:532: all] Error 2
make[1]: Leaving directory '/tmp/BUILD.fc40.x86_64/trinity-digikam-14.1.2~pre17+8f0b4a3a'
```
@Francois even though the FTBFS was already fixed by Slavek's code, I have added a second commit to the PR that removes old style unnecessary K&R function declarations. Could you test PR #24 one more time if you don't mind? It shouldn't cause any problem, but better to be sure.
@Francois even though the FTBFS was already fixed by Slavek's code, I have added a second commit to the PR that removes old style unnecessary K&R function declarations. Could you test PR #24 one more time if you don't mind? It shouldn't cause any problem, but better to be sure.
Basic information
Description
FTBFS
Steps to reproduce
Build
Screenshots
@Francois, please test patch from #24.
Thanks @SlavekB , new FTBFS below:
PR updated, please test it.
Thanks @SlavekB , new FTBFS below.
PR updated, other iterations can continue.
New FTBFS:
PR updated, other iterations can continue.
PR updated, other iterations can continue.
Nice @SlavekB , now digikam is built on Fedora 40. No more FTBFS.
Great, thank you for your excellent cooperation.
@Francois even though the FTBFS was already fixed by Slavek's code, I have added a second commit to the PR that removes old style unnecessary K&R function declarations. Could you test PR #24 one more time if you don't mind? It shouldn't cause any problem, but better to be sure.
The latest patch still works on Fedora 40. Thanks @MicheleC .
Thanks for the second test @Francois