Bug 245485

It seems you are not sponsored yet?

Perhaps. What do I need to do? I thought that Spot was going to sponsor this.

Spot, did you change the spec file? Then please
upload a new srpm.

Notes:
* Variables
  - Please check what %configure does and do not set unneeded
    variables.
    E.g. CXXFLAGS="$RPM_OPT_FLAGS": not needed
       CC="%{?ccache} gcc" CXX="%{?ccache} g++" also not needed
* Debuginfo
  - LDFLAGS="-s" is strictly forbidden

* Timestamps
  - Please add 'INSTALL="%{__install}" -p" option to
    'make install' to keep timestamps
  - And use "install -p" when using install command

* Seemingly unneeded scripts
  - When I do "make install", the following scripts are
    automatically done and seem unneeded
-------------------------------------------------------
#icons
install -m644 -D %{name}.png %{buildroot}/%{_datadir}/icons/%{name}.png
#tips file
install -m644 -D src/wxgui/tips.txt %{buildroot}/%{_datadir}/%{name}/fityk_tips.txt
--------------------------------------------------------

* scriptlets
  - Please explain why the call of update-mime-database is needed
    (Check the section "mimeinfo" of
     http://fedoraproject.org/wiki/Packaging/ScriptletSnippets )

* Requires
---------------------------------------------------------
Requires:	desktop-file-utils shared-mime-info
---------------------------------------------------------
  These two should be removed (these two seem to be added
  by spot)

  

Hi Tasaka,

Updated SRPM available at:
http://software.opensuse.org/download/home:/jdpipe/Fedora_7/src/fityk-0.8.1-16.1.src.rpm

I made the corrections you suggested to variables, scriptlets (mime database
update is not require at present, due to file association with *.fit being
removed), timestamps and debug info. 

The suggestions about the icon and tips file seems incorrect. I found that I
need at least the icon file, and as I recall, the tips file was also required,
due to having been omitted from the install phase in the current upstream release.

The current spec file is embedded in the src rpm.

Cheers
JP

Created attachment 157707 [details]
mock build log of fityk 0.8.1-16.1

Umm...
(In reply to comment #4)
> The suggestions about the icon and tips file seems incorrect. 
  - What is strange is that the build log actually says:
----------------------------------------------------------
 /usr/bin/install -p -m 644 'tips.txt'
'/var/tmp/fityk-0.8.1-16.1-buildroot/usr/share/fityk/tips.txt'
make[4]: Leaving directory `/builddir/build/BUILD/fityk-0.8.1/src/wxgui'
......
 /usr/bin/install -p -m 644 'fityk.png'
'/var/tmp/fityk-0.8.1-16.1-buildroot/usr/share/pixmaps/fityk.png'
make[2]: Leaving directory `/builddir/build/BUILD/fityk-0.8.1'
......
-----------------------------------------------------------
    So:
    - Actually fityk.png is installed. And
-----------------------------------------------------------
install -m644 -D %{name}.png %{buildroot}/%{_datadir}/icons/%{name}.png
-----------------------------------------------------------
      seems wrong because no package (on my system) installs
      icon png data under %{_datadir}/icons 
      (not %{_datadir}/icons/hicolor/??x??/apps) and %{_datadir}/pixmaps
      should be okay for the directory where pixmaps data is installed.
    - And
      tips.txt is also installed. However, the source expects:
      (in src/wxgui/gui.cpp)
-----------------------------------------------------
  1012	void FFrame::OnTipOfTheDay(wxCommandEvent&)
  1013	{
  1014	    string tip_file = "fityk_tips.txt";
  1015	    string tip_path = get_full_path_of_help_file(tip_file); 
-----------------------------------------------------
      the tips file named "fityk_tips.txt". So:
      - Perhaps the installed file "/usr/share/fityk/tips.txt" is
	not needed and should be removed (actually
	/usr/share/fityk/fityk_tips.txt and /usr/share/fityk/tips.txt
	are the same
       - or you should apply a patch against src/wxgui/gui.cpp.

* Documentation
  - "INSTALL" file is usually needed for people who want to
    rebuild this package by themselves and is not needed for people
    who want to install the package using rpm.
       - and please use "install -p" to keep timestamps
------------------------------------------------------------
install -m644 -D src/wxgui/tips.txt
%{buildroot}/%{_datadir}/%{name}/fityk_tips.txt
------------------------------------------------------------
	  when you want to use "install" command.

* Release number
  - please use integer for release number (some exceptions exist)
  - and you may want to use %{?dist} tag. While this is not
    required, it is preferable to avoid confusion.
    http://fedoraproject.org/wiki/Packaging/DistTag

Hi again

Here is an updated package:
http://software.opensuse.org/download/home:/jdpipe/Fedora_7/i386/fityk-0.8.1-18.2.i386.rpm

I removed the 'INSTALL' file, the duplicate tips file and the duplicate icon.

Note that the release numbers are automatically generated by the openSUSE Build
Service. I am happy for the %{?dist} thing to be added when the package is
finally accepted, however I can't easily add it during this test phase.

Cheers
JP

woops wrong URL. Should be:
http://software.opensuse.org/download/home:/jdpipe/Fedora_7/src/fityk-0.8.1-18.2.src.rpm

Created attachment 157717 [details]
gdb log of fityk

* Release number
> Note that the release numbers are automatically generated by 
> the openSUSE Build
> Service. 
  - Well, we are on Fedora, not on openSUSE... Please fix
    release number manually.
    And actually the %changelog entry says the version number
    is -8, not -18.1 (they must coincide, except for %?dist tag).

* For icon dir,
  - No.. What I said is that the icon should be under /usr/share/pixmaps,
    not under /usr/share/icons.

* Desktop category
  - Currently on GNOME desktop fityk entry appear in
    "その他” (this is Japanese, meaning "etc" in English) entry.
    IMO desktop categories for fedora-fityk.desktop should be:
----------------------------------------------------
Categories=Science;Education;GTK;
----------------------------------------------------

* gdb log
  - For me, fityk always causes double free corruption when:
    * launch filtk
    * choose Help-> About
    The gdb log is attached. Currently I don't know whether this
    is a bug of fityk side or wxGTK side or something else (GLib+?)

FYI my environ is:
gtk2-2.11.4-1.fc8
glib2-2.13.5-1.fc8
wxGTK-2.8.3-2.fc7
fityk-0.8.1-18.2

Hi, I'm upstream fityk maintainer

(In reply to comment #8)
> * gdb log
>   - For me, fityk always causes double free corruption when:
>     * launch filtk
>     * choose Help-> About
>     The gdb log is attached. Currently I don't know whether this
>     is a bug of fityk side or wxGTK side or something else (GLib+?)

I can't reproduce it, because I don't have gtk 2.11, but it's a bug in wx, that
was fixed in CVS a week ago:

revision 1.97
date: 2007/06/18 06:03:50;  author: MR;  state: Exp;  lines: +2 -2
gtk_border_free is for freeing GtkBorder's, not g_free.
Using g_free instead used to have no ill effects as gtk_border_free called that 
anyway, but in gtk+-2.11 GtkBorder
uses GSlice and gtk_border_free therefore uses g_slice_free and using g_free mak
es things crash hard.
So fix it or wxGTK won't work with the upcoming gtk+-2.12 stable release planned
 for end of July.

As a workaround, you may remove the line
bu_ok->SetDefault();
in fityk-0.8.1/src/wxgui/dialogs.cpp

I don't know if this problem will happen also in other places.

Marcin

Marcin, Thank you for useful information.
Then I think for now that free() corruption is not a bug of fityk.

Then, would you check my comment 8, John?

I think I addressed all of the remaining issues. Please try:
http://ascend.cheme.cmu.edu/ftp/fityk-0.8.1-8.fc6.src.rpm
http://ascend.cheme.cmu.edu/ftp/fityk.spec

Almost okay.

* Any reason you want to add %{?__cc:CC="%__cc"} %{?__cxx:CXX="%__cxx"} ?
* For desktop file:
--------------------------------------------------------
set -i 's/Categories=Application;Science/Categories=Science;Education/g'
fityk.desktop
--------------------------------------------------------
  It should be "sed".

BTW, will spot want to sponsor you?

Regarding the '__cc' thing, this was so that I can efficiently rebuild on my
slow and limited-memory FC6 machine at home. This way the rebuild is much
faster, as I set __cc to 'ccache gcc' (and likewise for c++) in my .rpmmacros.

The 'set' thing was wrong, as you discovered. Thanks for that. I corrected the
file on the server, but am not able to update the .src.rpm from here.
http://ascend.cheme.cmu.edu/ftp/fityk.spec

Well, then again is spot going to sponsor you?

Hi Tasaka,

Yes, Spot will sponsor me. I just heard from him. He has just returned from
holidays.

Cheers
JP

Okay.

----------------------------------------------
  This package (fityk) is APPROVED by me
----------------------------------------------

Please follow the procedure written on
http://fedoraproject.org/wiki/PackageMaintainers/Join
from "Get a Fedora Account"

If you want to push this package also on F-7, you
also have to check:
http://fedoraproject.org/wiki/Infrastructure/UpdatesSystem/Bodhi-info-DRAFT
after the URL above.

!! Well, recenctly Fedora package system changed a lot !!
   If you have some questions, please let us know.

ping?

I'm not sure what I'm supposed to do next. I have got a Fedora Account now, but
the instructions say that I need to create a Review Request, which AFAICT this
already is. What's next? My fedora account is 'jpye'.

Woops. Missed your preceding comments. I will work through those instructions.

Cheers
JP

If you have some questions, please let us know.

New Package CVS Request
=======================
Package Name: fityk
Short Description: Curve-fitting program
Owners: john, wojdyr
Branches: FC-6 F-7
InitialCC: john, wojdyr

Please can someone set the fedora-cvs flag to '?'. I can't do it, even though I
am a member of the fedora group 'fedorabugs'.

Once setting cvs to -.
wojdyr doesn't seem to be cvsextras member.

Hi Tasaka

That's fine. If he's not in cvsextras then leave him out. He can still be on the
CC list though, right?

Cheers
JP

(In reply to comment #24)

>  He can still be on the
> CC list though, right?

Okay. Please rewrite cvs request (and set cvs flag if possible)

New Package CVS Request
=======================
Package Name: fityk
Short Description: Curve-fitting program
Owners: john
Branches: FC-6 F-7
InitialCC: john, wojdyr

I have added fityk to the fedora CVS and have successfully build in the 'devel'
branch and it looks like it will build OK in F-7. I had an error message with
plague-client (below), but it looks like it is building now as well.

What do I need to do now? Do I close this bug?

-----
[john@jdpipe FC-6]$ plague-client list
Error: connection to the server timed out. '(110, 'Operation timed out.')'
[john@jdpipe FC-6]$ make build
/usr/bin/plague-client build fityk fityk-0_8_1-9_fc6 fc6
Package fityk enqueued.  Job ID: 35037.
[john@jdpipe FC-6]$ 

Yes. Follow the last step in the Contributor process documented here:

http://fedoraproject.org/wiki/PackageReviewProcess

Closed as NEXTRELEASE.