Bug 840551 - Review Request: sugar-kuku - arithmetic education game
Summary: Review Request: sugar-kuku - arithmetic education game
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2012-07-16 15:01 UTC by Danishka Navin
Modified: 2021-07-15 07:51 UTC (History)
8 users (show)

Fixed In Version: sugar-kuku-5-0.fc20
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2013-12-01 09:27:42 UTC
Type: ---
panemade: fedora-review+
gwync: fedora-cvs+

Attachments (Terms of Use)

Description Danishka Navin 2012-07-16 15:01:43 UTC
Spec URL: http://snavin.fedorapeople.org/packages/sugar-kuku/sugar-kuku.spec

SRPM URL: http://snavin.fedorapeople.org/packages/sugar-kuku/sugar-kuku-4-1.fc17.src.rpm

Kuku, short for 'Kuku Anakula' (Hungry Chicken), is based on the basic arithmetic education game Number Munchers. In single player mode, the game consists of a simple grid that the player (a chicken) is allowed to move around on. A question is posed outside of the grid, such as '5 + 5 = ?', and each square in the grid has a potential answer in it. The goal is to 'eat' as many correct answers as possible within a given time frame to accumulate points. There are penalties for eating incorrect answers, and the questions are changed when all correct answers are eaten. As the game progresses, the questions become more difficult. In multiplayer mode, the grid will be able to expand with multiple chickens that can compete against each other, or work as a team to eat all the correct answers. 


Fedora Account System Username: snavin

Comment 1 Vasiliy Glazov 2012-07-17 06:59:17 UTC

1. [!]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
     beginning of %install.

2. Add COPYING MANIFEST and README to %files section like %doc.

3. rpmlint error:
    sugar-kuku.noarch: E: script-without-shebang /usr/share/sugar/activities/KukuAnakula.activity/kuku_config.py

Please correct this and check with fedora-review.

Comment 2 Parag AN(पराग) 2012-07-17 16:33:22 UTC

+ koji scratch build ->http://koji.fedoraproject.org/koji/taskinfo?taskID=4246775

- rpmlint on rpms gave
sugar-kuku.noarch: W: no-documentation
sugar-kuku.noarch: E: script-without-shebang /usr/share/sugar/activities/KukuAnakula.activity/kuku_config.py
2 packages and 0 specfiles checked; 1 errors, 1 warnings.

+ Source verified with upstream as (sha1sum)
a424a2ed5995262aa88f2a926f06348f2b8f64fd  kuku_anakula-4.xo
a424a2ed5995262aa88f2a926f06348f2b8f64fd  ../SOURCES/kuku_anakula-4.xo

- License is GPLv3.

1) Change the license tag to GPLv3+

2) Remove the following lines from spec

3)Good to add documentation files as suggested in above comment.

4) fix the rpmlint error by using
in %prep.

Comment 4 Vasiliy Glazov 2012-07-18 05:50:33 UTC
Remove %defattr(-,root,root,-)

Comment 6 Parag AN(पराग) 2012-07-18 06:13:01 UTC
I can still see defattr in spec

Comment 7 Parag AN(पराग) 2012-07-18 06:13:33 UTC
also, start reviewing other packages. This will help you to get sponsorship soon.

Comment 8 Danishka Navin 2012-07-18 06:19:36 UTC
my bad forget to delete the entry but modify the changedlog



Comment 9 Danishka Navin 2012-07-18 06:48:42 UTC
do i need to run fedora-review on my system when I am reviewing a package?

i try to run the fedora-review on my own package but could not get an result.
looking for a guide.
i used the following format.

$fedora-review - <bug-number>

Comment 10 Vasiliy Glazov 2012-07-18 07:02:48 UTC
fedora-review -b <bug-number>

And read man fedora-review.

Comment 11 Parag AN(पराग) 2012-07-18 07:29:13 UTC
I am still not in support for any automated tool for a package review. Good to look package manually and learn the packaging by finding problems and suggesting solutions.

Comment 12 Parag AN(पराग) 2012-07-18 07:32:55 UTC
Recent update looks Ok.


Comment 13 Danishka Navin 2012-07-18 07:38:55 UTC
Thanks Parag
Btw, i was wondering how comment#10 like report produce?

Comment 14 Parag AN(पराग) 2012-07-18 07:46:03 UTC
why you revoked the fedora-review+ flag? Don't want me to review this package?

I have never used fedora-review.

Comment 15 Danishka Navin 2012-07-18 07:49:05 UTC
i think it was happened when we both commented at the same time.

i did not purposely change or revoked the fedora-review+ flag.

i was suppose to reply to your comment#11. Really sorry for the incontinence.

Comment 16 Parag AN(पराग) 2012-07-18 08:14:43 UTC
Ok. Please read http://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group#Reviewing_packages and do unofficial reviews.

Comment 17 Danishka Navin 2012-07-18 10:51:25 UTC
fedora‑cvs if not active for me and i can't set fedora-cvs flag to "?"

'If you are newly sponsored member of the Fedora Packager group you might have to wait an hour before you will get the permission to set the flag in Bugzilla as the sync is done hourly.'

i was waiting for few hours :)

Comment 18 Danishka Navin 2012-07-18 11:03:11 UTC
previous comment should be correct as follows

fedora‑cvs is not active for me and i can't set fedora-cvs flag to "?

Comment 19 Parag AN(पराग) 2012-07-18 13:33:52 UTC
  I have not yet sponsored you. That is why the next step I asked to do reviews. Once you do sufficient say 5 different in detail package reviews, I will sponsor you. That is reason I have not yet removed FE-NEEDSPONSOR flag from this bug.

Once you get sponsored, you will get email about that. Accepting review is different that getting sponsored. 

Read "Although FE-NEEDSPONSOR is set on the package's review request, it is the person that needs to be sponsored into the packager group, not the package. The package undergoes a review and approval that is separate from someone sponsoring you." from http://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group#Submitting_quality_new_packages

Comment 20 Danishka Navin 2012-07-18 13:52:18 UTC
i got a mail that saying 

'Thank you for applying for the packager group.

Welcome to the Fedora packager group. Please continue the process from: https://fedoraproject.org/wiki/PackageMaintainers/Join#Add_Package_to_Source_Code_Management_.28SCM.29_system_and_Set_Owner'

01. Do I need to set the FE-NEEDSPONSOR flag in each bug I create? I am keep packaging.

just now created the bug 841239

02. I try my best to work on reviews but my mail target was helping both Sugar and Fedora communities get most of activities packaged. :)
Since this not personally affecting me as this is just the way i spent my free time but not for my day job i may put less priority for reviewing.
But i do respect community formalities and the culture. :)

Thanks Parag & Vasiliy

Comment 21 Parag AN(पराग) 2012-07-18 14:08:30 UTC
The Sponsor who want to sponsor contributor need to add his fas name in packager group. As I will sponsor you, I have added your request for packager group and you got automatic email notification. When I will move this request to Sponsored then you will again get one more automatic email that you have been sponsored.

As you want to create more review requests and I decided to sponsor you provided you will do in detail full package review, I will remove FE-NEEDSPONSOR from existing reviews and you don't need to add it for other reviews. 

But if other reviews get approved in meantime, you still unable to request cvs flag for them.

So, Do some reviews and tell me here.

Comment 22 Danishka Navin 2012-07-18 16:21:50 UTC
anyway how these approved packages moving to build system?

and can i use the koji?

Comment 23 Danishka Navin 2012-07-18 16:23:46 UTC
and why I can't run fedora-review against this approved bug?

I have try to review my own package, just for testing.
But I do not get proper report.

This the result.

fedora-review -b 840551
Processing bugzilla bug: 840551
Bugzilla v0.7.0 initializing
Chose subclass RHBugzilla v0.1
Trying bugzilla cookies for authentication
Getting .spec and .srpm Urls from : 840551
  --> SRPM url: http://snavin.fedorapeople.org/packages/sugar-kuku/sugar-kuku-4-6.fc17.src.rpm
  --> Spec url: http://snavin.fedorapeople.org/packages/sugar-kuku/sugar-kuku.spec
Using review directory: /home/danishka/rpmbuild/840551-sugar-kuku
Downloading .spec and .srpm files
Downloading (Source0): http://mirrors.mit.edu/sugarlabs/activities/4526/kuku_anakula-4.xo
Running checks and generate report

Rebuilding /home/danishka/rpmbuild/840551-sugar-kuku/srpm/sugar-kuku-4-6.fc17.src.rpm using default root
ERROR: Exception(/home/danishka/rpmbuild/840551-sugar-kuku/srpm/sugar-kuku-4-6.fc17.src.rpm) Config(fedora-17-x86_64) 0 minutes 0 seconds
INFO: Results and/or logs in: /home/danishka/rpmbuild/840551-sugar-kuku/results
ERROR: Command failed. See logs for output.
Build failed rc = Build error(s)
Exception down the road...
[danishka@localhost rpmbuild]$ cat /home/danishka/rpmbuild/840551-sugar-kuku/results/
build.log  root.log   state.log 
[danishka@localhost rpmbuild]$ cat /home/danishka/rpmbuild/840551-sugar-kuku/results/build.log
Mock Version: 1.1.22
[danishka@localhost rpmbuild]$ cat /home/danishka/rpmbuild/840551-sugar-kuku/results/root.log
INFO backend.py:928:  Mock Version: 1.1.22
DEBUG backend.py:286:  rootdir = /var/lib/mock/fedora-17-x86_64/root/
DEBUG backend.py:287:  resultdir = /home/danishka/rpmbuild/840551-sugar-kuku/results
INFO backend.py:290:  calling preinit hooks
DEBUG util.py:57:  ensuring that dir exists: /var/cache/mock/fedora-17-x86_64/root_cache/
DEBUG util.py:307:  Executing command: ['tar', '--use-compress-program', 'pigz', '-xf', '/var/cache/mock/fedora-17-x86_64/root_cache/cache.tar.gz', '-C', '/var/lib/mock/fedora-17-x86_64/root/'] with env {'LANG': 'en_US.UTF-8', 'TERM': 'vt100', 'SHELL': '/bin/bash', 'HOSTNAME': 'mock', 'HOME': '/builddir', 'PATH': '/usr/bin:/bin:/usr/sbin:/sbin'}
DEBUG util.py:257:  pigz abort: corrupted input -- invalid deflate data: <stdin>
DEBUG util.py:257:  tar: Unexpected EOF in archive
DEBUG util.py:257:  tar: Unexpected EOF in archive
DEBUG util.py:257:  tar: Error is not recoverable: exiting now
DEBUG util.py:347:  Child return code was: 2
DEBUG util.py:111:  kill orphans
[danishka@localhost rpmbuild]$ cat /home/danishka/rpmbuild/840551-sugar-kuku/results/state.log
2012-07-18 19:33:05,146 - Mock Version: 1.1.22
2012-07-18 19:33:05,148 - State Changed: unpacking root cache

Comment 24 Parag AN(पराग) 2012-07-18 16:47:07 UTC
(In reply to comment #22)
> anyway how these approved packages moving to build system?

Once you raise fedora-cvs? flag, someone will create git module for your package in dist-git by changing flag to fedora-cvs+

Then you import the approved srpm in dist-git.

I think you better first read all the links here http://fedoraproject.org/wiki/PackageMaintainers/Join

then http://fedoraproject.org/wiki/Package_Review_Process

Comment 25 Parag AN(पराग) 2012-07-18 17:10:44 UTC
I ran fedora-review command on this bug. I used fedora-review first time and it executed successfully.

INFO: Done(/home/parag/Downloads/840551/sugar-kuku-4-6.fc17.src.rpm) Config(fedora-rawhide-i386) 16 minutes 25 seconds
INFO: Results and/or logs in: /var/lib/mock/fedora-rawhide-i386/result
State Changed: end
Build completed ok

Comment 26 Danishka Navin 2012-07-18 17:58:54 UTC
i know but was it was not working on my Fedora 17 x86_64 box?

i was using it as local user.

Comment 27 Danishka Navin 2012-07-18 19:18:50 UTC
if you can understand me correctly, 

is there a person who can move this to the build system other than myself ?

I won't available for next week but i want to get the work do.
Since we all work for the community and why the hell some one can do that on behalf of myself. 

I am not going to do any reviews within next two weeks as I have some other work to do. To be honest tomorrow morning I am going to a remote site, for a school lab project.

Just try to understand what I am saying.

Anyway I noticed that there is no such mandatory for doing 5 unofficial reviews and also some people did not any of review.

So why do you pushing back?

I do not want to add 'package manager' label to my profile but i want to contribute. 

Dear Parag,
Working for day job and working for the community spirit is different. :)

Comment 28 Parag AN(पराग) 2012-07-19 02:42:22 UTC
You are asking me to get you sponsored based that you don't have time to review other packages? What if all new people coming with this mindset. Then, we will endup with so many package reviews waiting. This reviewing work not only demonstrate that you understand rpm packaging well but help other people who used to wait years and years to get first initial comment on their package review.

So tell me who is pushing? There are reviews lying since years in Fedora and still some of them getting regularly updated but not yet finished.

There is no precise definition given for sponsorship by FESCo. So its upto sponsor to decide criteria for sponsorship and I can see you are still not familiar with recent fedora packaging changes. So I asked to do reviews and its also written in sponsorship policy.

You can even search in bugzilla that other sponsor not even approve the package before they can see contributor has done reviews.

So if you have no time in next 2 weeks then you can resume work after that. Else, find someone who is already in packager group and he can take over these reviews and request fedora-cvs and build it in fedora.

If you think my criteria of asking people to do 5 reviews is wrong then I will not proceed here and remove your packager request.

Please reply so that I can add back FE-NEEDSPONSOR in your all reviews so that other sponsor can look into your reviews and then sponsor you.

If I remember I didn't even sponsor my team members just because they need sponsorship and has not done any reviews. For me reviews are necessary to get sponsorship.

Comment 29 Parag AN(पराग) 2012-07-19 04:19:55 UTC
From http://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group#Submitting_quality_new_packages

"The sponsors that look at new package submissions often ask new packagers to do some package reviews in order to further show that they know what they're doing. If you do some reviews ahead of time, you can show the sponsors that you've both read these pages and understand the guidelines. Go ahead and link to other package review requests where you've left comments and reviews"

Here meaning of "some reviews" is not defined and is upto sponsor.

Comment 30 Danishka Navin 2012-07-19 05:36:24 UTC
i came with a spirit to work on packaging.. Specially had an idea of packaging all the activities for fedora.

but i feel.. i should give up this.

Comment 31 Parag AN(पराग) 2012-07-19 05:56:20 UTC
Whoever want to learn packaging, he will try different packages or review different kind of package like perl, python, gnome or library packages. I think if anyone knows well packaging then he can finish 5 reviews in whole day easily.

anyway, I consider above reply mean you don't want to review other packages.

Comment 32 Vasiliy Glazov 2012-07-20 05:53:30 UTC
Package Review

- = N/A
x = Pass
! = Fail
? = Not evaluated

==== Generic ====
[x]: EXTRA Rpmlint is run on all installed packages.
     Note: There are rpmlint messages (see attachment).
[x]: EXTRA Spec file according to URL is the same as in SRPM.
[x]: MUST Package is licensed with an open-source compatible license and meets
     other legal requirements as defined in the legal section of Packaging
[x]: MUST Package successfully compiles and builds into binary rpms on at
     least one supported primary architecture.
[x]: MUST %build honors applicable compiler flags or justifies otherwise.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
     that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Buildroot is not present
     Note: Unless packager wants to package for EPEL5 this is fine
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Package has no %clean section with rm -rf %{buildroot} (or
     Note: Clean would be needed if support for EPEL is required
[x]: MUST Sources contain only permissible code or content.
[x]: MUST Each %files section contains %defattr if rpm < 4.4
     Note: Note: defattr macros not found. They would be needed for EPEL5
[-]: MUST Macros in Summary, %description expandable at SRPM build time.
[-]: MUST Package contains desktop file if it is a GUI application.
[-]: MUST Development files must be in a -devel package
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Package complies to the Packaging Guidelines
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[x]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
     beginning of %install.
     Note: rm -rf would be needed if support for EPEL5 is required
[-]: MUST Large documentation files are in a -doc subpackage, if required.
[x]: MUST If (and only if) the source package includes the text of the
     license(s) in its own file, then that file, containing the text of the
     license(s) for the package is included in %doc.
[x]: MUST License field in the package spec file matches the actual license.
     Note: Checking patched sources after %prep for licenses. Licenses found:
     "*No copyright* UNKNOWN", "GPL (v3 or later)" For detailed output of
     licensecheck see file: /home/vascom/840551-sugar-kuku/licensecheck.txt
[x]: MUST Package consistently uses macro is (instead of hard-coded directory
[x]: MUST Package is named using only allowed ascii characters.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generate any conflict.
     Note: Package contains no Conflicts: tag(s)
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[x]: MUST Package is not relocatable.
[x]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint is run on all rpms the build produces.
     Note: No rpmlint messages.
[x]: MUST Sources used to build the package match the upstream source, as
     provided in the spec URL.
[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
[-]: MUST Package contains systemd file(s) if in need.
[x]: MUST File names are valid UTF-8.
[x]: SHOULD Reviewer should test that the package builds in mock.
[-]: SHOULD If the source package does not include license text(s) as a
     separate file from upstream, the packager SHOULD query upstream to
     include it.
[x]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
[?]: SHOULD Package functions as described.
[x]: SHOULD Latest version is packaged.
[x]: SHOULD Package does not include license text files separate from
[!]: SHOULD SourceX / PatchY prefixed with %{name}.
     Note: Source0 (kuku_anakula-4.xo)
[x]: SHOULD SourceX is a working URL.
[x]: SHOULD Description and summary sections in the package spec file contains
     translations for supported Non-English languages, if available.
[x]: SHOULD Package should compile and build into binary rpms on all supported
[-]: SHOULD %check is present and all tests pass.
[?]: SHOULD Packages should try to preserve timestamps of original installed
[x]: SHOULD Spec use %global instead of %define.

Checking: sugar-kuku-4-6.fc17.src.rpm
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

Rpmlint (installed packages)
Cannot parse rpmlint output:
sugar-kuku-4-6.fc17.noarch.rpm (rpmlib, GLIBC filtered):

    sugar-kuku = 4-6.fc17

MD5-sum check
http://mirrors.mit.edu/sugarlabs/activities/4526/kuku_anakula-4.xo :
  MD5SUM this package     : 05e0ed5cdb577bbe61341446e8bc5829
  MD5SUM upstream package : 05e0ed5cdb577bbe61341446e8bc5829

Generated by fedora-review 0.2.0 (53cc903) last change: 2012-07-09
Command line :/usr/bin/fedora-review -b 840551
External plugins:

Comment 34 Parag AN(पराग) 2012-08-09 03:59:24 UTC
Sorry I have lost interest in sponsoring you based on comment#27. I am revoking this package approval as its of no use if you have no packager group commit access. I will keep other package approvals for now.

Please find some other sponsor whom you can convince that how you are working for Fedora packages, its fixes and willing to actively maintain packages in Fedora.

Revoking this package approval is needed for me as it will let other sponsor to look your work and then approve this package again and accordingly sponsor you.

Another thing I want to note here from my side is that , I am still not in favour to use fedora-review command to post unoffical reviews where you are just posting the results and not even looking inside tarball and spec for any other issues.

Please add FE-NEEDSPONSOR in your other package submissions.

Comment 35 Steven Dake 2012-08-13 22:08:46 UTC
Regarding fedora-review tool results came from as part of a package sponsorship, I validate sponsored applicants on their ability to use fedora review tool, and actually review the package by examining tarball and spec and have a running conversation with the original packager.

I agree with you though, if folks are doing only fedora-review tool running without examining the spec file manually and conversing with the unofficial package reviewer, that is not sufficient.

Comment 36 Parag AN(पराग) 2012-08-14 03:13:00 UTC
Thanks for your comment. I was of the same opinion. I have also reported upstream developers of fedora-review that such things are happening by using fedora-review tool.  They assured me that they are not going to market this tool for new contributors easy way to get sponsorship. They are also thinking to add some comment at the start of auto generated review text file.

Comment 37 Danishka Navin 2012-08-14 09:09:29 UTC
Dear Parag and Steven,

I am not here to do master in reviewing or put my free time on reviewing.

I was suppose to package sugar activities for fedora which are having same structure. 

Please try to understand what i was trying to and i do not want to contribute as a reviewer as I do not have much free time.

Due to this reviewing thing all packaged sugar activities are on hold and i do not have nothing to do.

And next thing is other sugar activity packages did not follow this 5 reviews rule.

You guys should think again about how to get people in to reviewing but not forcing others to review to reduce the number of pending reviews.

I have packaged more than 8  sugar activities but i did not continue after this incident.

I do not work on each and every application  but i just want to stick in to sugar activities.

Comment 38 Parag AN(पराग) 2012-08-14 09:15:56 UTC
(In reply to comment #37)
> Dear Parag and Steven,
> I am not here to do master in reviewing or put my free time on reviewing.
> I was suppose to package sugar activities for fedora which are having same
> structure. 
> Please try to understand what i was trying to and i do not want to
> contribute as a reviewer as I do not have much free time.
> Due to this reviewing thing all packaged sugar activities are on hold and i
> do not have nothing to do.
> And next thing is other sugar activity packages did not follow this 5
> reviews rule.
Note: again this is my criteria to get sponsored from me. I already said there are may packager sponsors in Fedora. Try to contact any one of them and they if like your work will sponsor you.

> You guys should think again about how to get people in to reviewing but not
> forcing others to review to reduce the number of pending reviews.

If people think like this then as I said already we will end up with so many package reviews waiting for their initial comment at least.

> I have packaged more than 8  sugar activities but i did not continue after
> this incident.
> I do not work on each and every application  but i just want to stick in to
> sugar activities.

If you have no time they why not ask on devel list to either work on these packages or submit these packages again with fresh review request with new owners?

Comment 39 Danishka Navin 2012-08-14 09:22:36 UTC
I am serious, we should find a way hire new volunteers for reviewing.

I will write to ambassadors list.

We must try to increase our resource base.

Comment 40 Jason Tibbitts 2012-08-14 12:07:05 UTC
Somehow I got CC'd on this.  I'm not really sure why, but a couple of points:

1) I've been on vacation for a month.  For longer the lifetime of this package.  I'm not really back to reviewing packages yet.

2) I don't know anything about sugar, so I'm not sure there's much I can do.  I prefer to stick to things I understand when sponsoring new contributors.

There was a sugar package from a different contributor where I offered to sponsor if I could get a sugar expert to join in, and couldn't even get that.  To me it seems like this is a failing with the sugar community, not really the package review process.  It's true we have a number of waiting packages, but the number is pretty much stable.  Besides, this one isn't really even that old so I'm not sure what's up with the grumbling.

If someone's going to pay for reviewers, that's great, but some of us have been reviewing a large number of packages for years without compensation and I don't really think money would help the process.

Finally, I personally dispensed with requiring people to review other packages in order to receive sponsorship some time ago.  Sometimes it makes sense, sometimes it doesn't.  But each sponsor is allowed to sponsor according to their own personal beliefs, so I'm certainly not going to say that my way is somehow better.

Comment 41 Parag AN(पराग) 2012-08-14 12:27:01 UTC
Thanks for your comment. Sorry for not mentioning why I cc'ed you. I know that you have done a HUGE work(contribution) to Fedora and its Package Review process and policies. 

also, if you remember few times when I got some issues in review process, I used to talk to you on IRC specially when topics used to come up for minimizing package review tickets.

Now reason to CC you is in case you may want to say anything on this review progress which you already said.


Comment 42 Danishka Navin 2013-06-04 06:24:24 UTC

Do I need to add python2-devel here?
Btw, there is no update in the upstream since I packaged this. I hope there is no issue with this package. :)

Comment 43 Parag AN(पराग) 2013-06-04 06:37:46 UTC
Got the request from Danishka again for this review as he has been sponsored now. So, someone will be there for taking care of his work from Sugar group. 

  Yes. Check if upstream is providing python2 or python3 files and then accordingly add versioned Buildrequires. Like for this package python2-devel. And remove BuildRequires: python.

Please provide again SPEC and SRPM for final look on this package.

Comment 45 Parag AN(पराग) 2013-06-06 06:31:43 UTC

+ Package builds in mock successfully

+ rpmlint on rpms gave
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

+ Source verified with upstream as (sha256sum)
srpm tarball : adbfda234c4c5d31c0acd25b65e42d50cd9baacd1e171f5bcc4d96a3438fa308
upstream tarball : adbfda234c4c5d31c0acd25b65e42d50cd9baacd1e171f5bcc4d96a3438fa308

+ License tag is valid GPLv3+

+ Rest looks as per packaging guidelines.

1)  Don't use any backslash between %{buildroot} and %{_prefix}, so your %install should look like
%{__python} ./setup.py install --prefix=%{buildroot}%{_prefix}

2) Why is changelog for -6 and -7 is same? Add correct changelog

3) You don't need to set again executable bits. remove following from spec
#chmod +x  kuku_config.py

Comment 46 Parag AN(पराग) 2013-08-06 05:22:25 UTC
 Can you fix above issues and submit a new srpm?

Comment 47 Danishka Navin 2013-08-06 05:27:59 UTC

Comment 49 Parag AN(पराग) 2013-08-06 06:34:57 UTC
spec file inside srpm does not match with above spec file.

Comment 51 Parag AN(पराग) 2013-08-06 08:46:35 UTC
koji scratch build failed for f20, http://koji.fedoraproject.org/koji/taskinfo?taskID=5783085

you need to have correct BR: sugar-toolkit and not sugar-toolkit-gtk3

Also, add correct changelog like what got changed from previous srpm release.

Comment 52 Danishka Navin 2013-08-06 08:58:47 UTC
Hi Parag,

I am using F18 box for packaging. There is no BR:sugar-toolkit package 

error: Failed build dependencies:
	BR:sugar-toolkit is needed by sugar-kuku-4-7.fc18.noarch

Comment 53 Parag AN(पराग) 2013-08-06 09:02:47 UTC
ah! I mean use it like this

BuildRequires: sugar-toolkit

Comment 55 Parag AN(पराग) 2013-08-06 09:30:14 UTC

Comment 56 Danishka Navin 2013-08-06 09:35:59 UTC
New Package SCM Request
Package Name: sugar-kuku
Short Description: arithmetic education game
Owners: snavin
Branches: f17 f18 f19 f20

Comment 57 Christopher Meng 2013-08-06 10:04:57 UTC
SCM is invalid. 

Please drop f17 and f20.


Comment 58 Danishka Navin 2013-08-06 10:16:31 UTC
New Package SCM Request
Package Name: sugar-kuku
Short Description: arithmetic education game
Owners: snavin
Branches: f18 f19

Comment 59 Gwyn Ciesla 2013-08-06 13:05:27 UTC
Git done (by process-git-requests).

Comment 60 Fedora Update System 2013-08-07 01:45:52 UTC
sugar-kuku-4-8.fc18 has been submitted as an update for Fedora 18.

Comment 61 Fedora Update System 2013-08-07 01:46:05 UTC
sugar-kuku-4-8.fc19 has been submitted as an update for Fedora 19.

Comment 62 Fedora Update System 2013-08-07 22:56:00 UTC
sugar-kuku-4-8.fc19 has been pushed to the Fedora 19 testing repository.

Comment 63 Parag AN(पराग) 2013-08-26 09:14:50 UTC
Can this be pushed to stable now and closed this review?

Comment 64 Fedora Update System 2013-08-26 22:26:46 UTC
sugar-kuku-4-8.fc19 has been pushed to the Fedora 19 stable repository.

Comment 65 Fedora Update System 2013-08-26 22:27:15 UTC
sugar-kuku-4-8.fc18 has been pushed to the Fedora 18 stable repository.

Comment 66 Fedora Update System 2013-11-19 09:10:19 UTC
sugar-kuku-5-0.fc18 has been submitted as an update for Fedora 18.

Comment 67 Fedora Update System 2013-11-19 09:10:37 UTC
sugar-kuku-5-0.fc19 has been submitted as an update for Fedora 19.

Comment 68 Fedora Update System 2013-11-19 09:10:47 UTC
sugar-kuku-5-0.fc20 has been submitted as an update for Fedora 20.

Comment 69 Fedora Update System 2013-11-19 21:47:13 UTC
Package sugar-kuku-5-0.fc20:
* should fix your issue,
* was pushed to the Fedora 20 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing sugar-kuku-5-0.fc20'
as soon as you are able to.
Please go to the following url:
then log in and leave karma (feedback).

Comment 70 Fedora Update System 2013-12-01 09:27:42 UTC
sugar-kuku-5-0.fc18 has been pushed to the Fedora 18 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 71 Fedora Update System 2013-12-01 09:32:01 UTC
sugar-kuku-5-0.fc19 has been pushed to the Fedora 19 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 72 Fedora Update System 2013-12-14 02:48:28 UTC
sugar-kuku-5-0.fc20 has been pushed to the Fedora 20 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 73 JuliaFields 2020-12-16 10:39:34 UTC Comment hidden (spam)
Comment 74 Esther 2021-07-15 07:51:03 UTC Comment hidden (spam)

Note You need to log in before you can comment on or make changes to this bug.