6
0
Fork 0

cmake conversion #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.
Ghost hat das Label PR/wip vor 6 Jahren hinzugefügt
SlavekB hat vor 6 Jahren kommentiert
Besitzer

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:
Ghost hat vor 6 Jahren kommentiert
Ersteller

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.
SlavekB hat vor 6 Jahren kommentiert
Besitzer

Everything seems to be ready to merge.

Everything seems to be ready to merge.
SlavekB hat das Label PR/wip vor 6 Jahren entfernt
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
Der Pull Request wurde als 5c55403336 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/knmap#2
Laden…
Hier gibt es bis jetzt noch keinen Inhalt.