cmake conversion #1

Sloučený
SlavekB sloučil 1 commity z větve feat/cmakeConv do větve master před před 6 roky
Ghost okomentoval před 6 roky

I've set the install of the Russian keyboard as default since I believe those speakers are the main users in this apps.

I didn't define USE_BOLD_MENUITEM due to comment in boldmenuitem.h

I've set the install of the Russian keyboard as default since I believe those speakers are the main users in this apps. I didn't define USE_BOLD_MENUITEM due to comment in boldmenuitem.h
Ghost přidal/a PR/rfc štítek před 6 roky
MicheleC okomentoval před 6 roky
Vlastník

I think the default should be english, as for the whole TDE. Russian users can then change their settings

I think the default should be english, as for the whole TDE. Russian users can then change their settings
SlavekB okomentoval před 6 roky
Vlastník

Using USE_BOLD_MENUITEM is really "interesting" when the appropriate CPP file starts #undef USE_BOLD_MENUITEM. You're right, we obviously do not need this option.

The fixed definition of TDE_VERSION_MAJOR in config.h is incorrect. This value is defined in tdeversion.h in tdelibs.

The message "xkbfile is requested ..." in ConfigureChecks.cmake would probably be "xkbfile is required ...".

The definition WITH_RU_UA_LAYOUT is inconsistent. Either should be in the "optional stuff" section and be set by WITH_ALL_OPTIONS or should be named BUILD_RU_UA_LAYOUT. In my opinion, the first option is better. By the way, for automake, the default was "disabled". For CMake, it might also be the default OFF.

Using USE_BOLD_MENUITEM is really "interesting" when the appropriate CPP file starts `#undef USE_BOLD_MENUITEM`. You're right, we obviously do not need this option. The fixed definition of TDE_VERSION_MAJOR in `config.h` is incorrect. This value is defined in `tdeversion.h` in tdelibs. The message "xkbfile is requested ..." in `ConfigureChecks.cmake` would probably be "xkbfile is required ...". The definition WITH_RU_UA_LAYOUT is inconsistent. Either should be in the "optional stuff" section and be set by WITH_ALL_OPTIONS or should be named BUILD_RU_UA_LAYOUT. In my opinion, the first option is better. By the way, for automake, the default was "disabled". For CMake, it might also be the default OFF.
Ghost okomentoval před 6 roky
Autor

@Michele, I meant, I've set to install the Russian layout in the default install as Slavek have pointed out this layout is built as an option with automake.

@Michele, I meant, I've set to install the Russian layout in the default install as Slavek have pointed out this layout is built as an option with automake.
MicheleC okomentoval před 6 roky
Vlastník

@Greg: no problem, mine was just a comment from a "not-well-informed" general point of view.

I will leave it to you and Slavek to work out the details 😉

@Greg: no problem, mine was just a comment from a "not-well-informed" general point of view.<br> I will leave it to you and Slavek to work out the details :wink:
SlavekB okomentoval před 6 roky
Vlastník

I did squash @cethyel commits for cmake conversion and added two small ones. The scripts for tdeconf_update were not installed as programs => executable bit was not set during installation. At the same time, these scripts in source code have set executable bit unnecessarily.

I did squash @cethyel commits for cmake conversion and added two small ones. The scripts for tdeconf_update were not installed as programs => executable bit was not set during installation. At the same time, these scripts in source code have set executable bit unnecessarily.
SlavekB odstranil/a PR/rfc štítek před 6 roky
SlavekB uzavřel/a tento požadavek na natažení před 6 roky
SlavekB odstranil/a větev feat/cmakeConv před 6 roky
SlavekB přidal/a toto do milníku R14.0.6 release před 6 roky
Požadavek na natažení byl sloučen jako 03a86cbd51.
Přihlaste se pro zapojení do konverzace.
Žádní posuzovatelé
Bez milníku
Bez zpracovatelů
3 účastníků
Oznámení
Termín dokončení

Žádný termín dokončení.

Závislosti

Nejsou nastaveny žádné závislosti.

Reference: TDE/kkbswitch#1
Načítá se…
Není zde žádný obsah.