Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: geda-gschem - Electronics schematics editor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=204598 cgoorah(a)yahoo.com.au changed: What |Removed |Added ---------------------------------------------------------------------------- BugsThisDependsOn|204168 |204423 -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: geda-gschem - Electronics schematics editor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=204598 cgoorah(a)yahoo.com.au changed: What |Removed |Added ---------------------------------------------------------------------------- BugsThisDependsOn| |204168 -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: geda-gschem - Electronics schematics editor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=204598 cgoorah(a)yahoo.com.au changed: What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO| |204601 nThis| | -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: geda-gschem - Electronics schematics editor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=204598 mtasaka(a)ioa.s.u-tokyo.ac.jp changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody(a)fedoraproject.org |mtasaka(a)ioa.s.u-tokyo.ac.jp OtherBugsDependingO|163776 |163778 nThis| | -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: geda-gschem - Electronics schematics editor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=204598 ------- Additional Comments From mtasaka(a)ioa.s.u-tokyo.ac.jp 2006-08-31 11:48 EST ------- First review of geda-gschem : 1. From http://fedoraproject.org/wiki/Packaging/Guidelines : * Naming - Maybe http://www.geda.seul.org/tools/gschem/index.html is more suitable for URL . * Use rpmlint - E: geda-gschem devel-dependency libgeda-devel E: geda-gschem explicit-lib-dependency libgeda-devel E: geda-gschem explicit-lib-dependency libstroke-devel Please explain why this package (geda-gschem) should depend on libstroke-devel and libgeda-devel. * BuildRequires: - gtk2-devel, guile-devel, gd-devel <- required by libgeda-devel - gettext-devel ... is this really required? - mesa-libGL-devel ... also, is this really required? .... by my mock build, gettext-devel and mesa-libGL-devel seems unnecessary. * Encoding - /usr/share/doc/geda-gschem-20060123/AUTHORS is encoded in ISO-8859-1 and includes non-ascii character(s). Change the encoding to UTF-8. * Desktop files - Well, according to http://fedoraproject.org/wiki/Packaging/ScriptletSnippets , update-desktop-database seems to be needed when desktop file includes mimetype key. * Using %{buildroot} and %{optflags} vs $RPM_BUILD_ROOT and $RPM_OPT_FLAGS - Mixed use is found. Please unify the usage. * Timestamps - Well, this package have many text files and xpm files so keeping timestamps is desirable. Again, try to keep timestamps (does this package accept 'make INSTALL="install -p" install' ?) * File and Directory Ownership Umm... this is somewhat complicated..... Please recheck the owning issue of directories. - the directory %{_datadir}/gEDA/bitmap is owned by both geda-gsymcheck and geda-gschem. - geda-gsymcheck has only the directory and does not have any files under the directory. So the entry %dir %{_datadir}/gEDA/bitmap can be removed from geda-gsymcheck. - Or, if the trick written above is not desirable, the owner of %dir %{_datadir}/gEDA/bitma should be geda-symbols as both geda-gsymcheck and geda-gschem requires geda-symbols. 2. From http://fedoraproject.org/wiki/Packaging/ReviewGuidelines : = Nothing. 3. Other things I have noticed: = Nothing. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: geda-gschem - Electronics schematics editor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=204598 Bug 204598 depends on bug 204263, which changed state. Bug 204263 Summary: Review Request: geda-symbols - Electronic symbols for gEDA https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=204263 What |Old Value |New Value ---------------------------------------------------------------------------- Resolution| |NEXTRELEASE Status|ASSIGNED |CLOSED -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: geda-gschem - Electronics schematics editor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=204598 ------- Additional Comments From mtasaka(a)ioa.s.u-tokyo.ac.jp 2006-09-02 08:07 EST ------- Second review for geda-gschem. 1. From http://fedoraproject.org/wiki/Packaging/Guidelines : * Encoding - iconv -f EUCJP -t UTF8 AUTHORS.tmp > AUTHORS && \ Well... not EUCJP but ISO-8859-1 at this case. * Desktop files - Well: fedora-gschem.desktop has the entry: Icon=geda However, no icons are included in this package. Also, geda.png is included in geda rpm. If you want to use this png file, this bug must be blocked by bug 204259 and Requires: geda is needed. Perhaps you don't want this. So + Copy some png (or xpm) file and include it in this package. ( or you can use xpm file included in this package ). + Install the image file into /usr/share/icons/hicolor/...... as gschem.{png,xpm} (or use link) + Call gtk-update-icon-cache on %post and %postun. = File and Directory Ownership is now solved. 2. From http://fedoraproject.org/wiki/Packaging/ReviewGuidelines : = Nothing. 3. Other things I have noticed: = Nothing. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: geda-gschem - Electronics schematics editor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=204598 ------- Additional Comments From mtasaka(a)ioa.s.u-tokyo.ac.jp 2006-09-02 09:17 EST ------- Well: * As for Source2: Please specify the URL from which this png file can be gained, and if you did some action (like image type conversion) to create this png file, explain it briefly. * install -m 644 examples/*.sch %{buildroot}%{_datadir}/gEDA/examples install -m 644 examples/README.* %{buildroot}%{_datadir}/gEDA/examples install -m 644 scheme/*.scm %{buildroot}%{_datadir}/gEDA/scheme Use "install -p" Well, I want to recheck this once more. Please upload the new spec and srpm and I will check them (perhaps it is a final check). -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: geda-gschem - Electronics schematics editor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=204598 ------- Additional Comments From mtasaka(a)ioa.s.u-tokyo.ac.jp 2006-09-03 04:11 EST ------- Created an attachment (id=135454) --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=135454&action=...) gdb log of gschem Umm... 20060123-6 seems to cause SEGV. Seems 100% reproducible by the following steps. 1. cp /usr/share/gEDA/examples/adders_1.sch to current directory. 2. execute "gschem ./adders_1.sch" 3. Choose "File -> Quit" - Then gschem causes SEGV. I attach gdb log of this. Can you see the reason, or will you contact the upstream about this? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: geda-gschem - Electronics schematics editor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=204598 ------- Additional Comments From mtasaka(a)ioa.s.u-tokyo.ac.jp 2006-09-04 11:20 EST ------- Umm...??? Upstream closed this bug? Upstream says: This is the slice memory bug when running with gtk+ 2.8 or later. Fixed in new release of gEDA/gaf 20060824. Please upgrade -Ales .... What is gEDA/gaf? What component (released as Fedora Extras rpm) corresponds to gEDA/gaf? I would want to try it if it really fixes this SEGV problem. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: geda-gschem - Electronics schematics editor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=204598 ------- Additional Comments From cgoorah(a)yahoo.com.au 2006-09-04 16:50 EST ------- In case, mirrors haven't yet been updated: http://chitlesh.funpic.de/srpm -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: geda-gschem - Electronics schematics editor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=204598 cgoorah(a)yahoo.com.au changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=204598 Kevin Fenzi <kevin(a)tummy.com&gt; changed: What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+ --- Comment #16 from Kevin Fenzi <kevin(a)tummy.com&gt; 2008-12-22 22:23:06 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.