cmake conversion #2

Samengevoegd
SlavekB heeft 1 commits samengevoegd van feat/cmakeConv naar master 6 jaren geleden
Er is nog geen inhoud.
Ghost voegde het PR/wip label 6 jaren geleden toe
Eigenaar

Probably inadvertently, some files for CMake had a set executable flag. Corrected.

At the same time I prepared patches for TQT_NO_COMPAT and TQT_NO_ASCII_CAST. It's nice to see that turning these definitions off signifies that something is wrong in the code 😄

Probably inadvertently, some files for CMake had a set executable flag. Corrected. At the same time I prepared patches for TQT_NO_COMPAT and TQT_NO_ASCII_CAST. It's nice to see that turning these definitions off signifies that something is wrong in the code :smile:
Poster

I did notice the flags on these 3 files when I committed, git displayed them with chmod 755 and then I asked myself, Is it normal behavior? Let's push, see what Slavek will say.

It must have happen because I used these files to build the apps, I'll change that for a copy next time or check for that execute bit.

I did notice the flags on these 3 files when I committed, git displayed them with chmod 755 and then I asked myself, Is it normal behavior? Let's push, see what Slavek will say. It must have happen because I used these files to build the apps, I'll change that for a copy next time or check for that execute bit.
Eigenaar

Everything seems to be ready to merge.

Everything seems to be ready to merge.
SlavekB verwijderde het PR/wip label 6 jaren geleden
SlavekB heeft deze pull request gesloten 6 jaren geleden
Eigenaar

Merged and pushed upstream.

Merged and pushed upstream.
SlavekB heeft 6 jaren geleden de branch feat/cmakeConv verwijderd.
SlavekB heeft dit 6 jaren geleden aan de mijlpaal R14.0.6 release toegevoegd
De pull request is samengevoegd als 5c55403336.
Log in om deel te nemen aan deze discussie.
Geen beoordelaars
Geen mijlpaal
Niet toegewezen
2 deelnemers
Notificaties
Vervaldatum

Geen vervaldatum ingesteld.

Afhankelijkheden

Geen afhankelijkheden ingesteld.

Referentie: TDE/knmap#2
Laden…
Er is nog geen inhoud.