[Bug 167974] Review Request: hugs98 - Haskell Interpreter

[bugzilla at redhat.com]

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: hugs98 - Haskell Interpreter


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=167974


petersen at redhat.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |MODIFIED
OtherBugsDependingO|163778                      |163779
              nThis|                            |




------- Additional Comments From petersen at redhat.com  2006-04-21 21:38 EST -------
(In reply to comment #8)
> I would like to leave it as it is. Better too many requirements than
> not enough. Maybe there is a bug somewhere but building in mock
> without requiring xorg-x11-proto-devel did not succeed.
> Also, the build may seem to succeed, and the packages installed, but
> the binary modules may not be there! I had quite a fun time to
> figure out the requirements.

Hmm ok, it seemed to build fine for me in mock without them,
though I　didn't do an careful rpmdiff comparison...  I guess some
the early modular X packages had some missing -devel deps,
but I think that is all sorted out now.  Otherwise please report
the problem in bugzilla.

> > Also better to keep openal-devel around for when HOpenal gets updated
> > (ghc-6.4.2 includes a newer version apparently).  And it builds on fc4
> > anyway.
> The problem is, as it seems, that the openal package is then included
> but the binary module is not. I will include a commented out
> buildreq.

Ah, good point.  I suppose one could patch to the latest version released with
ghc-6.4.2.  Anyway it may be an idea to subpackage openal too for FC4 say:
so perhaps a %_with_openal switch would be useful.

(In reply to comment #9)
> > Btw what is "touch src/stamp-h.in" needed for?
> It appears that this is necessary so that configure creates
> all the necessary files. Without it, the prefix is set to
> "/usr/local" for example.

Oh, sounds like a bug then.  In my Fedora Haskell package
I didn't do that, but hadn't noticed any problems, I wasn't
using %configure - dunno if that is related.

> I have made some little changes, same file, same place.

Thanks for your submission and all your work on this.
The package is ACCEPTED.

Please build it for FE4 too. :-)

-- 
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.