6
0
Fork 0

conversion to cmake building system #2

Zusammengeführt
SlavekB hat 1 Commits von feat/cmakeConv nach master vor 6 Jahren zusammengeführt
Ghost hat vor 6 Jahren kommentiert
Hier gibt es bis jetzt noch keinen Inhalt.
SlavekB hat vor 6 Jahren kommentiert
Besitzer

I think that would be helpful in CMake conversions always define BUILD_ALL and WITH_ALL_OPTIONS to have the same standard across all modules.

Do you want to make an ammend of the previous commit, or should I push a separate commit to add these variables?

I think that would be helpful in CMake conversions always define `BUILD_ALL` and `WITH_ALL_OPTIONS` to have the same standard across all modules. Do you want to make an ammend of the previous commit, or should I push a separate commit to add these variables?
Ghost hat vor 6 Jahren kommentiert
Ersteller

Consistency is nice, I like that!

Since there's no need for options beside gcc visibility I went for the simplest.

If you have the time this evening, please commit, otherwise, I'll do It tomorrow.
@+

Consistency is nice, I like that! Since there's no need for options beside gcc visibility I went for the simplest. If you have the time this evening, please commit, otherwise, I'll do It tomorrow. @+
SlavekB hat vor 6 Jahren kommentiert
Besitzer

The TIME_WITH_SYS_TIME test in automake verifies whether time.h can be included together with sys/time.h. That's why I adjusted the cmake test to the same way.

Furthermore, I added a test of the existence of the strtoll function.

The TIME_WITH_SYS_TIME test in automake verifies whether `time.h` can be included together with `sys/time.h`. That's why I adjusted the cmake test to the same way. Furthermore, I added a test of the existence of the `strtoll` function.
SlavekB hat diesen Pull-Request vor 6 Jahren geschlossen
SlavekB hat vor 6 Jahren kommentiert
Besitzer

Merged and pushed upstream.

Merged and pushed upstream.
SlavekB löschte die Branch feat/cmakeConv vor 6 Jahren
SlavekB hat diesen Issue vor 6 Jahren zum R14.0.6 release Meilenstein hinzugefügt
Ghost hat vor 6 Jahren kommentiert
Ersteller

Including time.h and sys/time.h is convenient, I didn't know that could be done, nice!

Regarding testing functions, do we already have a test for each functions that are provided by the c++ lib or the c lib?

Including time.h and sys/time.h is convenient, I didn't know that could be done, nice! Regarding testing functions, do we already have a test for each functions that are provided by the c++ lib or the c lib?
SlavekB hat vor 6 Jahren kommentiert
Besitzer

Because for some reason HAVE_STRTOLL was used in the code, it was strange to define it in config.h without performing the test. I could either spend time researching whether this definition is still meaningful or add the test. I used a simpler solution 😄

Because for some reason `HAVE_STRTOLL` was used in the code, it was strange to define it in `config.h` without performing the test. I could either spend time researching whether this definition is still meaningful or add the test. I used a simpler solution :smile:
Der Pull Request wurde als ac46856010 gemergt.
Anmelden, um an der Diskussion teilzunehmen.
Keine Reviewer
Kein Meilenstein
Niemand zuständig
2 Beteiligte
Nachrichten
Fällig am

Kein Fälligkeitsdatum gesetzt.

Abhängigkeiten

Keine Abhängigkeiten gesetzt.

Referenz: TDE/tdeio-ftps#2
Laden…
Hier gibt es bis jetzt noch keinen Inhalt.