Conversion to the cmake building system. #3

已合并
SlavekB 4 年前 将 4 次代码提交从 feat/cmakeConv 合并至 master
Ghost 评论于 4 年前
这个人很懒,什么都没留下。
Ghost4 年前 添加了标签 PR/wip
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.
SlavekB4 年前 修改标题 WIP:Conversion to the cmake building system.Conversion to the cmake building system.
SlavekB4 年前 删除了标签 PR/wip
SlavekB4 年前 合并了提交 b3a20cdb91master
SlavekB4 年前 删除了分支 feat/cmakeConv
SlavekB4 年前 添加了里程碑 R14.0.9 release
该合并请求已作为 b3a20cdb91 被合并。
登录 并参与到对话中。
无审核者
未选择里程碑
未指派成员
2 名参与者
通知
到期时间

未设置到期时间。

依赖工单

没有设置依赖项。

参考:TDE/tde-systemsettings#3
正在加载...
这个人很懒,什么都没留下。