Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 1301219 - Review Request: racket - Racket is a full-spectrum programming language
Summary: Review Request: racket - Racket is a full-spectrum programming language
Status: CLOSED DUPLICATE of bug 1540833
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
: 808350 (view as bug list)
Depends On:
TreeView+ depends on / blocked
Reported: 2016-01-22 22:00 UTC by Brandon Thomas
Modified: 2018-02-12 16:55 UTC (History)
11 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2018-02-12 16:55:58 UTC

Attachments (Terms of Use)

Description Brandon Thomas 2016-01-22 22:00:28 UTC
Spec URL:
Description: Racket is a full-spectrum programming language. It goes beyond
Lisp and Scheme with dialects that support objects, types, laziness,
and more. Racket enables programmers to link components written
in different dialects, and it empowers programmers to create new,
project-specific dialects. Racket's libraries support applications from
web servers and databases to GUIs and charts.

Koji Build:

Fedora Account System Username:

This is my first package, and I need a sponsor. I though it was strange that Fedora doesn't include Racket, so I built it and I can maintain it to the best of my ability.

Comment 1 Neal Gompa 2016-01-22 22:34:43 UTC
Could you please put it somewhere that is easily accessible?

For example, you could use Copr to upload the SRPM and have it build there, and then go ahead and link us to the Spec in dist-git and the SRPM in the repo directory.

Also, your FAS user is not your email address. Please provide the correct FAS username.

Comment 2 Zbigniew Jędrzejewski-Szmek 2016-01-23 00:10:27 UTC
*** Bug 808350 has been marked as a duplicate of this bug. ***

Comment 3 Brandon Thomas 2016-01-23 15:29:31 UTC
Oh, sorry!

Fedora Account System Username: bthomas

Spec URL:

I ran the build on Copr, and it fails for rawhide, but works for Fedora 21, 22 and 23. I haven't found any related bugs upstream. I'm going to try to drill down on this some more to create a patch and isolate what changed in rawhide.
Build Log (rawhide):

Comment 4 Neal Gompa 2016-01-23 18:21:47 UTC
Now that I can see your spec, I see a few things you can correct pretty quickly.

* You do not need to do the export CFLAGS bit, as %configure handles that.

* %configure instead of %{configure}. I'm a bit hazy on this, but I think using %{configure} can mess up some circumstances (but I'm not sure). In any case, it makes stylistic sense to use %configure there anyway.

* %make_build instead of make %{?_smp_mflags}. Unless you are targeting EPEL with this, you should use this instead, as it's cleaner and more obvious what it is. (Actually, I believe %make_build now works in EL7 at least, so you could probably get away with using it even there). 

* %make_install instead of "make install %{?_smp_mflags} DESTDIR=${RPM_BUILD_ROOT}". Unless there's a very good reason to, you should use %make_install.

* Unless you are targeting EL5, you do not need a %clean section.

* Instead of using ExclusiveArch, use "ExcludeArch: %{arm}", as you indicate that only ARM is failing right now. Generally, you should only exclude architectures that don't work, rather than making it only build on a subset of architectures.

Comment 5 Upstream Release Monitoring 2016-01-23 22:26:52 UTC
bthomas's scratch build of racket-6.3-1.fc23.src.rpm for f24 completed

Comment 7 Neal Gompa 2016-01-24 08:48:00 UTC
You don't need "rm -rf ${RPM_BUILD_ROOT}" in the install step. rpmbuild automatically cleans the build root before starting %install.

Other than that, I don't see anything standing out to me.

Comment 8 Neal Gompa 2016-01-24 08:49:41 UTC
Also, adding FE-NEEDSPONSOR, since this is your first package review.

Comment 10 Brandon Thomas 2016-01-25 16:38:50 UTC
I based the above spec off of a previous attempt, and it has the same breakdown as before (racket and racket-devel), but I think I can break the package down a bit better. Because this is often used as an introductory language in higher education, I want to try to make this as non-confusing as possible to new programmers. So I propose the following breakdown:

racket-core: Core Racket components
racket-gtk: Graphical Racket components (requires: racket-core)
racket-doc: Racket documentation
racket-devel: Libraries to link with Racket
racket-ide: The DrRacket IDE for Racket (requires: racket-gtk)
racket: The complete Racket distribution (requires: racket-ide, racket-doc)

It is backwards compatible with previous attempts at adding Racket to Fedora (including the spec files above). I decided against calling "racket-ide" as "drracket", because lots of people think that DrRacket is the name of the entire distribution, and they might install that thinking they're installing everything, only to get mad when the documentation feature of drracket doesn't work (and as a new programmer who isn't familiar with Linux, it can be frustrating).
AFAIK, there isn't a list of packages that require graphical libraries, but here's what I can see off of the bat (usr/share/racket/pkgs): draw, draw-lib, future-visualizer, future-visualizer-typed, games, gui, gui-lib, gui-pkg-manager-lib, images-gui-lib (I want to keep some images functionality for servers running racket, if it doesn't break anything), pict (unfortunately I don't think I can break off just the gui component), pict-doc, pict-lib, pict-snip, picturing-programs, plot-gui-lib (again, I want to keep some plot functionality if possible), rackunit-gui, redex-gui-lib, slideshow, slideshow-exe, slideshow-lib, slideshow-plugin. And for DrRacket: drracket, drracket-plugin-lib, drracket-tool.

What to people think? Should I keep it the same as before, or should I try to break it down like this?

Comment 11 Jens Petersen 2017-01-04 03:21:10 UTC
I am completely new to racket, but overall your approach sounds reasonable.
It would probably be ideal if interdependencies (Requires) could be
I would suggest experiementing with the packaging first in Copr.

For others: most current copr currently seems to be:

Comment 12 Jens Petersen 2017-01-04 03:40:14 UTC
However it might be better to first get the current package into Fedora. :)

While it is good you have plenty of comments, it makes the rpm fields/headers
at the top of the spec file a bit hard to read.

It would be good to do another koji scratch build since aarch64 and ppc64le
have been added as new archs to F26 Rawhide.

Comment 13 Saša Janiška 2017-03-05 16:07:59 UTC
Racket language does deserve, imho, to be present in the distribution like Fedora, so please help to make it happen!

Comment 14 David Benoit 2017-07-06 14:13:55 UTC
Has there been any progress on this?  I'd love to see Racket in the Fedora repositories.  Is there anything I can do do help move this along?  

I've never submitted a package for review before.  If I start a new rpm build with updated sources, should I open a new Bugzilla report, or post here?

Comment 15 Damian Wrobel 2018-02-12 13:25:34 UTC
*** Bug 1540833 has been marked as a duplicate of this bug. ***

Comment 16 Damian Wrobel 2018-02-12 16:55:58 UTC

*** This bug has been marked as a duplicate of bug 1540833 ***

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