Conversion to the cmake building system. #3

Злито
SlavekB злито 4 комітів з feat/cmakeConv до master 4 роки тому
Ghost прокоментував(ла) 4 роки тому
Тут ще немає жодного змісту.
Ghost додано PR/wip з міткою 4 роки тому
Ghost прокоментував(ла) 4 роки тому
Автор

@SlavekB when you've got time can you have a look into the symbols visibility?

@SlavekB when you've got time can you have a look into the symbols visibility?
SlavekB прокоментував(ла) 4 роки тому
Власник

@SlavekB when you've got time can you have a look into the symbols visibility?

There seems to be no need for any additional care regarding hidden visibility, as the result is a standalone application without additional libraries and modules.

> @SlavekB when you've got time can you have a look into the symbols visibility? There seems to be no need for any additional care regarding hidden visibility, as the result is a standalone application without additional libraries and modules.
SlavekB рецензовано 4 роки тому
SlavekB додав коментар
Власник

It looks good. Here are some small comments.

It looks good. Here are some small comments.
##### other data
install(
SlavekB прокоментував(ла) 4 роки тому
Власник

These are also desktop style files for which we will want to use desktop file translation using TWTW. Therefore, it is also possible to use tde_create_translated_desktop here.

These are also desktop style files for which we will want to use desktop file translation using TWTW. Therefore, it is also possible to use `tde_create_translated_desktop` here.
DESTINATION ${XDG_APPS_INSTALL_DIR}
)
install(
SlavekB прокоментував(ла) 4 роки тому
Власник

These are also desktop style files for which we will want to use desktop file translation using TWTW. Therefore, it is also possible to use tde_create_translated_desktop here.

These are also desktop style files for which we will want to use desktop file translation using TWTW. Therefore, it is also possible to use `tde_create_translated_desktop` here.
SlavekB прокоментував(ла) 4 роки тому
Власник

Note: I have a patch for ascii-cast ready – it's single line.

Note: I have a patch for ascii-cast ready – it's single line.
Ghost прокоментував(ла) 4 роки тому
Автор

the desktop files installation have been added to the cmake conversion.

the desktop files installation have been added to the cmake conversion.
SlavekB прокоментував(ла) 4 роки тому
Власник
  • Fixed and added the use of tde_create_translated_desktop.
  • Added commit to resolve ascii-cast.
  • The first step of moving translations to a new layout has been performed.
+ Fixed and added the use of tde_create_translated_desktop. + Added commit to resolve ascii-cast. + The first step of moving translations to a new layout has been performed.
SlavekB прокоментував(ла) 4 роки тому
Власник

I wanted to move forward and merge it, because everything looks good, but subsequently I noticed that in tde-packaging for Debian there is a man page. Will you add it to the main tree?

I wanted to move forward and merge it, because everything looks good, but subsequently I noticed that in tde-packaging for Debian there is a man page. Will you add it to the main tree?
SlavekB прокоментував(ла) 4 роки тому
Власник

One more note: Because the project name is set to tde-systemsettings, the translation catalogs are now installed as tde-systemsettings.mo.

This is basically not a problem – on the contrary, it can be an advantage that it is not a very common name. However, there will need to be an adjustment in systemsettings/main.cpp for TDELocale::setMainCatalogue(...) to set tde-systemsettings instead of the current systemsettings.

What is your opinion – keep the new name of the catalog or revert to the previous one?

One more note: Because the project name is set to `tde-systemsettings`, the translation catalogs are now installed as `tde-systemsettings.mo`. This is basically not a problem – on the contrary, it can be an advantage that it is not a very common name. However, there will need to be an adjustment in `systemsettings/main.cpp` for `TDELocale::setMainCatalogue(...)` to set `tde-systemsettings` instead of the current `systemsettings`. What is your opinion – keep the new name of the catalog or revert to the previous one?
Ghost прокоментував(ла) 4 роки тому
Автор

...I noticed that in tde-packaging for Debian there is a man page...

Man page added, please test.

>...I noticed that in tde-packaging for Debian there is a man page... Man page added, please test.
Ghost прокоментував(ла) 4 роки тому
Автор

What is your opinion – keep the new name of the catalog or revert to the previous one?

Looks to me that a thorough job should change the name/ocurrence: systemsettings vs tde-systemsettings everywhere in this module; namely code, config and desktop files alike, help and man page...
It shouldn't be difficult but time consumming and I don't feel like doing It. 😩

> What is your opinion – keep the new name of the catalog or revert to the previous one? Looks to me that a thorough job should change the name/ocurrence: **systemsettings** vs **tde-systemsettings** everywhere in this module; namely code, config and desktop files alike, help and man page... It shouldn't be difficult but time consumming and I don't feel like doing It. :weary:
SlavekB прокоментував(ла) 4 роки тому
Власник

...I noticed that in tde-packaging for Debian there is a man page...

Man page added, please test.

Excellent, tested, everything is fine.

> >...I noticed that in tde-packaging for Debian there is a man page... > > Man page added, please test. > > Excellent, tested, everything is fine.
SlavekB прокоментував(ла) 4 роки тому
Власник

What is your opinion – keep the new name of the catalog or revert to the previous one?

Looks to me that a thorough job should change the name/ocurrence: systemsettings vs tde-systemsettings everywhere in this module; namely code, config and desktop files alike, help and man page...
It shouldn't be difficult but time consumming and I don't feel like doing It. 😩

Ok, that's not a problem – we can move the translations to be installed as systemsettings.mo, as it was before.

> > What is your opinion – keep the new name of the catalog or revert to the previous one? > > Looks to me that a thorough job should change the name/ocurrence: **systemsettings** vs **tde-systemsettings** everywhere in this module; namely code, config and desktop files alike, help and man page... > It shouldn't be difficult but time consumming and I don't feel like doing It. :weary: Ok, that's not a problem – we can move the translations to be installed as `systemsettings.mo`, as it was before.
SlavekB змінився заголовок з WIP:Conversion to the cmake building system. на Conversion to the cmake building system. 4 роки тому
SlavekB видалено PR/wip з міткою 4 роки тому
SlavekB Злиті коміти b3a20cdb91 в master 4 роки тому
SlavekB видалена гілка feat/cmakeConv 4 роки тому
SlavekB додав(ла) до R14.0.9 release етапу 4 роки тому
Запит на злиття був влитиий як b3a20cdb91.
Підпишіться щоб приєднатися до обговорення.
Немає рецензентів
Етап відсутній
Немає виконавця
2 учасників
Сповіщення
Дата завершення

Термін виконання не встановлений.

Залежності

No dependencies set.

Reference: TDE/tde-systemsettings#3
Завантаження…
Тут ще немає жодного змісту.