Initial commit of ibus-mozc ( T317 )
I tested on KDE Plasma Desktop and GNOME Desktop.
We're not using any of the WTL bits, this can be removed.
This is only referenced in the installer, which we don't use. Not needed.
This is only referenced in the installer (which we don't use) and the unicode files don't mention a specific Unicode DFS license.
I didn't see any reference to Unlicense in the codebase, where is this from?
Did you hand write your package.yml? We have a script to generate it: There's no other way you would've made this mistake, so seems like you need to correct all this and resubmit / update your diff: https://getsol.us/articles/packaging/packaging-practices/en/#generating-a-package-yml
Eliminate the unnecessary new lines.
All of your install commands should be combined like -Dm00644 or -Dm00755. Not to mention you should be using a loop over a list of files instead of line-by-line, file-by-file. This applies to all the images as well, so that needs to be corrected.
I considering about license. ( I wrote at T317#159801 )
We can see mozc's third party license at https://github.com/google/mozc/blob/master/src/data/installer/credits_en.html
'Okinawa dictionary' was licensed by Public License(Unlicense?) and 'WTL' was licensed by MS-PL.
I searched other distribution's mozc package and found they listed only licenses included in SPDX or show "BSD3-License and custom".
Therefore I listed licenses included in SPDX.
How should I change licenses' list ? ( I have no experience of treating such a package include many third party code. Would you give me advicese?)
As explained in my review, there are items which are only referenced in the credits / installer which we do not use or aren't relevant to us. The "Public Domain" referenced for the Okinawa is strictly in Japanese and is not a license which is identified by SPDX, thus shouldn't be listed. There is no guarantee they actually mean an identified public domain or "unlicense" license.
As mentioned in my previous review:
So your installation of all these libs needs to be corrected.
Should I replace 'ibus-mozc' and 'mozc.xml' install commands with loops?
I think replacing them makes redundancy and needless, so I didn't replace them.
But if I should, I will replace.