https://bugzilla.redhat.com/show_bug.cgi?id=827723
Bug ID: 827723 QA Contact: extras-qa@fedoraproject.org Severity: medium Version: rawhide Priority: medium CC: notting@redhat.com, package-review@lists.fedoraproject.org Assignee: nobody@fedoraproject.org Summary: Review Request: gnuhealth - The free Health and Hospital Information System Regression: --- Story Points: --- Classification: Fedora OS: Linux Reporter: michel+fdr@sylvestre.me Type: --- Documentation: --- Hardware: All Mount Type: --- Status: NEW Component: Package Review Product: Fedora
Spec URL: http://salimma.fedorapeople.org/specs/healthcare/gnuhealth.spec SRPM URL: http://salimma.fedorapeople.org/specs/healthcare/gnuhealth-1.4.5-1.fc17.src.... Description: GNU HEALTH is a free, centralized Health and Hospital Information System that provides the following functionality :
* Electronic Medical Record (EMR) * Hospital Information System (HIS) * Health Information System
The goal is to collaborate with health professionals around the world to improve the quality of life of the underprivileged, providing a free system that optimizes health promotion and disease prevention.
Fedora Account System Username: salimma
https://bugzilla.redhat.com/show_bug.cgi?id=827723
--- Comment #1 from Michel Alexandre Salim michel+fdr@sylvestre.me --- Note that gnuhealth 1.6.1 is out, but can't be packaged yet until the Tryton stack is updated to the latest 2.4 series -- this is the last release in the 1.4.x line compatible with Tryton 2.2.x
https://bugzilla.redhat.com/show_bug.cgi?id=827723
Michel Alexandre Salim michel+fdr@sylvestre.me changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends On| |827722
https://bugzilla.redhat.com/show_bug.cgi?id=827723
Bug 827723 depends on bug 827722, which changed state.
Bug 827722 Summary: Review Request: python-qrcode - Python QR Code image generator https://bugzilla.redhat.com/show_bug.cgi?id=827722
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Resolution|--- |DUPLICATE
https://bugzilla.redhat.com/show_bug.cgi?id=827723
Michel Alexandre Salim michel+fdr@sylvestre.me changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends On| |824628
https://bugzilla.redhat.com/show_bug.cgi?id=827723
--- Comment #2 from Michel Alexandre Salim michel+fdr@sylvestre.me --- Question for reviewer: since gnuhealth as it is basically is a set of modules for the Tryton application framework, it might be more appropriate to name it trytond-gnuhealth and make it virtually provide gnuhealth. Thoughts?
https://bugzilla.redhat.com/show_bug.cgi?id=827723
Bug 827723 depends on bug 827722, which changed state.
Bug 827722 Summary: Review Request: python-qrcode - Python QR Code image generator https://bugzilla.redhat.com/show_bug.cgi?id=827722
What |Removed |Added ---------------------------------------------------------------------------- Status|CLOSED |ASSIGNED Resolution|DUPLICATE |---
https://bugzilla.redhat.com/show_bug.cgi?id=827723
Bug 827723 depends on bug 824628, which changed state.
Bug 824628 Summary: Review Request: python-qrcode - Pure python QR Code generator https://bugzilla.redhat.com/show_bug.cgi?id=824628
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution|--- |DUPLICATE
https://bugzilla.redhat.com/show_bug.cgi?id=827723
Dan Horák dan@danny.cz changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |dan@danny.cz
--- Comment #3 from Dan Horák dan@danny.cz --- (In reply to comment #2)
Question for reviewer: since gnuhealth as it is basically is a set of modules for the Tryton application framework, it might be more appropriate to name it trytond-gnuhealth and make it virtually provide gnuhealth. Thoughts?
I think the package should be called gnuhealth as it is a standalone application, even when it uses Tryton as its runtime environment.
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=827723
Shakthi Kannan shakthimaan@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |shakthimaan@gmail.com Assignee|nobody@fedoraproject.org |shakthimaan@gmail.com
--- Comment #4 from Shakthi Kannan shakthimaan@gmail.com --- Can you please update this to 1.8.1?
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=827723
--- Comment #5 from Michel Alexandre Salim michel+fdr@sylvestre.me --- Will do. Could you set the review flag?
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=827723
Shakthi Kannan shakthimaan@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-review?
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=827723
--- Comment #6 from Shakthi Kannan shakthimaan@gmail.com --- A 1.8.1-1 release is now available.
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=827723
--- Comment #7 from Shakthi Kannan shakthimaan@gmail.com --- Can you please provide an update on this?
https://bugzilla.redhat.com/show_bug.cgi?id=827723
--- Comment #8 from Shakthi Kannan shakthimaan@gmail.com --- Ping!
https://bugzilla.redhat.com/show_bug.cgi?id=827723
Michel Alexandre Salim michel+fdr@sylvestre.me changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(shakthimaan@gmail | |.com)
--- Comment #9 from Michel Alexandre Salim michel+fdr@sylvestre.me --- I unfortunately don't have the time at the moment to package this - new dependencies keep getting added (right now, trytond-stock_lot) and given the number of manual steps needed to get an installation going, I have a bit too much on my plate.
Shakthi, if you'd like to take over, I'd be happy to review this and any other dependency you need to submit as new packages. Apologies for the delay!
https://bugzilla.redhat.com/show_bug.cgi?id=827723
Shakthi Kannan shakthimaan@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEW Flags|fedora-review? | Flags|needinfo?(shakthimaan@gmail | |.com) |
--- Comment #10 from Shakthi Kannan shakthimaan@gmail.com --- Will look into it.
https://bugzilla.redhat.com/show_bug.cgi?id=827723
Shakthi Kannan shakthimaan@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Assignee|shakthimaan@gmail.com |nobody@fedoraproject.org
https://bugzilla.redhat.com/show_bug.cgi?id=827723
--- Comment #11 from Shakthi Kannan shakthimaan@gmail.com --- Updated to gnuhealth-1.6.1 and tested on Fedora 18:
SPEC: http://shakthimaan.fedorapeople.org/SPECS/gnuhealth.spec SRPM: http://shakthimaan.fedorapeople.org/SRPMS/gnuhealth-1.6.1-1.fc18.src.rpm
Successful Koji build for F18:
http://koji.fedoraproject.org/koji/taskinfo?taskID=5579265
https://bugzilla.redhat.com/show_bug.cgi?id=827723
antonio montagnani antonio.montagnani@alice.it changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |antonio.montagnani@alice.it
--- Comment #12 from antonio montagnani antonio.montagnani@alice.it --- how do you start the program?? Downloaded from fedorapeople page
https://bugzilla.redhat.com/show_bug.cgi?id=827723
--- Comment #13 from Shakthi Kannan shakthimaan@gmail.com --- You will need to install, setup trytond, tryton and postgresql using the installation documentation at:
http://en.wikibooks.org/wiki/GNU_Health/Installation
https://bugzilla.redhat.com/show_bug.cgi?id=827723
Yaniv Bronhaim bronhaim@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |bronhaim@gmail.com
--- Comment #14 from Yaniv Bronhaim bronhaim@gmail.com --- Hey, did an informal review and found the following issues: 1. instead of repeating "gnuhealth" hard-coded in the spec, you should use %{name} - as you do with %{version}
2. when you use python source you should require BuildRequires: python2-devel or python3-devel
https://bugzilla.redhat.com/show_bug.cgi?id=827723
Christopher Meng cickumqt@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |cickumqt@gmail.com
--- Comment #15 from Christopher Meng cickumqt@gmail.com --- gnuhealth 2.2.2 is available.
https://bugzilla.redhat.com/show_bug.cgi?id=827723
--- Comment #16 from Christopher Meng cickumqt@gmail.com --- (In reply to Shakthi Kannan from comment #13)
You will need to install, setup trytond, tryton and postgresql using the installation documentation at:
This should be done by the package IMO.
https://bugzilla.redhat.com/show_bug.cgi?id=827723
--- Comment #17 from Matthias Runge mrunge@redhat.com --- (In reply to Christopher Meng from comment #16)
(In reply to Shakthi Kannan from comment #13)
You will need to install, setup trytond, tryton and postgresql using the installation documentation at:
This should be done by the package IMO.
Christopher, please keep in mind, any change in data by just installing a package is explicitly not desired. Since you can not know a database password, this wouldn't even be possible. And configuration of packages is a main difference between .dep and .rpm packages.
https://bugzilla.redhat.com/show_bug.cgi?id=827723
gil cattaneo puntogil@libero.it changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |201449 (FE-DEADREVIEW)
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=201449 [Bug 201449] FE-DEADREVIEW -- Reviews stalled due to lack of submitter response should be blocking this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=827723
--- Comment #18 from Germano Massullo germano.massullo@gmail.com --- Any news?
https://bugzilla.redhat.com/show_bug.cgi?id=827723
Bastián Díaz diazbastian@openmailbox.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |diazbastian@openmailbox.org
--- Comment #19 from Bastián Díaz diazbastian@openmailbox.org --- Wow ... I surprised me when I found out I was not yet in the repositories. I usually use this software a lot and is very important in their area, however, installation is not intuitive for a common user in Workstation.
You could be a candidate for Flatpak?
https://bugzilla.redhat.com/show_bug.cgi?id=827723
Jun Aruga jaruga@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |jaruga@redhat.com
--- Comment #20 from Jun Aruga jaruga@redhat.com --- How do we fix this?
Unfortunately, below links are "403 Forbidden".
SPEC: http://shakthimaan.fedorapeople.org/SPECS/gnuhealth.spec SRPM: http://shakthimaan.fedorapeople.org/SRPMS/gnuhealth-1.6.1-1.fc18.src.rpm
https://bugzilla.redhat.com/show_bug.cgi?id=827723
--- Comment #21 from Ankur Sinha (FranciscoD) sanjay.ankur@gmail.com --- May be best to close this and open up a fresh review.
To everyone above - if you're interested in using the package on Fedora, please consider packaging it up and helping with maintenance.
Cheers! Ankur
https://bugzilla.redhat.com/show_bug.cgi?id=827723
Ankur Sinha (FranciscoD) sanjay.ankur@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Resolution|--- |WONTFIX Last Closed| |2018-01-07 17:47:28
--- Comment #22 from Ankur Sinha (FranciscoD) sanjay.ankur@gmail.com --- Closing as WONTFIX.
package-review@lists.fedoraproject.org