Bug 1141557

hi!

There are some issues:

[ ] Package contains BR: python2-devel or python3-devel

And from rpmlint:

Checking: sugar-dimensions-56-1.fc22.noarch.rpm
          sugar-dimensions-56-1.fc22.src.rpm
sugar-dimensions.noarch: W: non-standard-group Sugar/Activities
sugar-dimensions.noarch: E: non-executable-script /usr/share/sugar/activities/Dimensions.activity/gencards.py 0644L /usr/bin/env
sugar-dimensions.noarch: E: non-executable-script /usr/share/sugar/activities/Dimensions.activity/helpbutton.py 0644L /usr/bin/env
sugar-dimensions.noarch: E: incorrect-fsf-address /usr/share/sugar/activities/Dimensions.activity/dimensions.py
sugar-dimensions.noarch: E: invalid-lc-messages-dir /usr/share/locale/aym/LC_MESSAGES/org.sugarlabs.VisualMatchActivity.mo
sugar-dimensions.src: W: non-standard-group Sugar/Activities
sugar-dimensions.src:14: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 14)
2 packages and 0 specfiles checked; 4 errors, 3 warnings.

Please fix this and mention the renaming in %changelog.

Cheers,
 Florian

Hi Florian,

Warning of non-standard-group is common to all sugar activities that are being packaged.

The rest of the issues are fixed.

Here are the new files,
Spec URL: https://callkalpa.fedorapeople.org/sugar-dimensions/sugar-dimensions.spec
SRPM URL: https://callkalpa.fedorapeople.org/sugar-dimensions/sugar-dimensions-56-2.fc20.src.rpm

Hi Kalpa,

what about the errors, already mentioned?

[ ] non-executable-script /usr/share/sugar/activities/Dimensions.activity/gencards.py 0644L /usr/bin/env

[ ] non-executable-script /usr/share/sugar/activities/Dimensions.activity/helpbutton.py 0644L /usr/bin/env

[ ] incorrect-fsf-address /usr/share/sugar/activities/Dimensions.activity/dimensions.py

[ ] invalid-lc-messages-dir /usr/share/locale/aym/LC_MESSAGES/org.sugarlabs.VisualMatchActivity.mo

[ ] mention the renaming in %changelog

[ ] %files section contains %defattr if rpm < 4.4
    Note: %defattr present but not needed
    https://fedoraproject.org/wiki/Packaging:Guidelines#File_Permissions

[ ] Package requires other packages for directories it uses.
     Note: No known owner of /usr/share/locale/pis/LC_MESSAGES,
     /usr/share/locale/pis, /usr/share/locale/mvo/LC_MESSAGES,
     /usr/share/locale/pbs, /usr/share/locale/quz, /usr/share/locale/quy,
     /usr/share/locale/hus, /usr/share/locale/pbs/LC_MESSAGES,
     /usr/share/locale/quz/LC_MESSAGES, /usr/share/locale/fa_AF/LC_MESSAGES,
     /usr/share/locale/ayc/LC_MESSAGES, /usr/share/locale/fa_AF,
     /usr/share/locale/quy/LC_MESSAGES, /usr/share/locale/tzo/LC_MESSAGES,
     /usr/share/locale/hus/LC_MESSAGES, /usr/share/locale/aym,
     /usr/share/locale/ayc, /usr/share/locale/aym/LC_MESSAGES,
     /usr/share/locale/mvo, /usr/share/locale/tzo, /usr/share/locale/tzm,
     /usr/share/locale/tzm/LC_MESSAGES

[ ] Package must own all directories that it creates.
     Note: Directories without known owners: /usr/share/locale/quz,
     /usr/share/locale/ayc/LC_MESSAGES, /usr/share/locale/pis/LC_MESSAGES,
     /usr/share/locale/pis, /usr/share/locale/tzo, /usr/share/locale/tzm,
     /usr/share/locale/fa_AF/LC_MESSAGES, /usr/share/locale/mvo/LC_MESSAGES,
     /usr/share/locale/tzm/LC_MESSAGES, /usr/share/locale/fa_AF,
     /usr/share/locale/quy/LC_MESSAGES, /usr/share/locale/aym,
     /usr/share/locale/ayc, /usr/share/locale/aym/LC_MESSAGES,
     /usr/share/locale/quy, /usr/share/locale/hus,
     /usr/share/locale/pbs/LC_MESSAGES, /usr/share/locale/mvo,
     /usr/share/locale/pbs, /usr/share/locale/hus/LC_MESSAGES,
     /usr/share/locale/tzo/LC_MESSAGES, /usr/share/locale/quz/LC_MESSAGES

[ ] Package contains BR: python2-devel or python3-devel
    Instead of just removing python-devel you should take a look at
    https://fedoraproject.org/wiki/Packaging:Python#BuildRequires

Cheers,
 Florian

Hi Florian,

My bad, I ran rpmlint only on the srpm but not on the rpm.

I think I already had python-devel as a BR. I am waiting for the upstream to fix the FSF address issue. Will update once it is fixed.

hi Kalpa,

As https://fedoraproject.org/wiki/Packaging:Python#BuildRequires says, you have to distinguish between python2-devel and python3-devel.

Cheers,
 Florian

hi Kalpa,

are you still interessted in a review?

Cheers,
 Florian

It's been many years since an update here, but... are you still interested in this package?

Note that you don't *have* to wait for upstream to fix FSF address issues; it's enough to report them.

Anyway, as per the "stalled review policy" https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews please respond soon if you're still interested in this ticket, otherwise I will mark it FE-DEADREVIEW and close it.

Please mark this ticket as FE-DEADREVIEW. Thank you.

Fair enough. Flagging as DEADREVIEW and closing.