[Bug 868263] Review Request: compat-guile1.8 - Compatibility package for guile-1.8

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=868263

--- Comment #8 from Jaroslav Škarvada <jskarvad@xxxxxxxxxx> ---
Jan thanks for the update.

In the srpm there was spec file with the different name (compat-guile1.8.spec),
I used the standalone spec (compat-guile18).

Some more issues I noticed:

compat-guile18.spec:126: W: mixed-use-of-spaces-and-tabs (spaces: line 126,
tab: line 79)
compat-guile18.src: E: description-line-too-long C Install the compat-guile18
package if you'd like to add extensibility to programs
> It should be easy fix.

I noticed the upgrade path is broken - I installed guile/devel packages to
rawhide and tried upgrading to guile2/compat/devel packages. The upgrade
failed. I think there shouldn't be guile2 package (because we already have
compat-guile18) and guile-2 should be used instead.

I don't know much about compat packages but from the guide [1] about renaming:
>$provEVR refers to an (Epoch-)Version-Release tuple the original unchanged package would have had if it had been version or release bumped
In rawhide there is guile-1.8.8-6, so the compat-guile should probably use
5:%{version}-7
I think the devel subpackage should also have the provides, obsoletes. But in
all cases the upgrade path should work.

>I think these are needed in the runtime, try (require-extension (srfi 60)).
Mirek, thanks for pointing this out. You are right, they are needed. It would
be great to have comment about it in the spec.

[1]
http://fedoraproject.org/wiki/Packaging:Guidelines#Renaming.2FReplacing_Existing_Packages

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=J5ax6jMigD&a=cc_unsubscribe
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]