[spin-kickstarts] Scientific: including sage fails depcheck for no clear reason

[Amit Saha]

commit 8f0b1a1f68709622e9b38260ea501bd486f18ac0
Author: Amit Saha <asaha(a)redhat.com&gt;
Date:   Tue Sep 10 13:00:57 2013 +1000

    Scientific: including sage fails depcheck for no clear reason

 fedora-livedvd-scientific-kde.ks |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)
---
diff --git a/fedora-livedvd-scientific-kde.ks b/fedora-livedvd-scientific-kde.ks
index 2578ecd..f1199e0 100644
--- a/fedora-livedvd-scientific-kde.ks
+++ b/fedora-livedvd-scientific-kde.ks
@@ -26,8 +26,9 @@ scilab-devel
 scilab-doc
 
 # sagemath
-sagemath
-sagemath-notebook
+# TBD: need to figure out why this fails
+#sagemath
+#sagemath-notebook
 
 #Devel tools

[Bruno Wolff III]

On Tue, Sep 10, 2013 at 03:01:08 +0000,
   Amit Saha <amitksaha(a)fedoraproject.org&gt; wrote:
> commit 8f0b1a1f68709622e9b38260ea501bd486f18ac0
> Author: Amit Saha <asaha(a)redhat.com&gt;
> Date:   Tue Sep 10 13:00:57 2013 +1000
>
>     Scientific: including sage fails depcheck for no clear reason 
The error message explains what happened, though it's a bit unusual.
sagemath requires /bin/sh and no package appears to provide that. 
Either bash should provide both /bin/sh and /usr/bin/sh or sagemath 
should require /usr/bin/sh instead of /bin/sh. There was some discussion 
about this problem in general on one of the lists roughly a month ago. 
Probably the best way to start is by filing a bug with sagemath.

Also note, that you won't see this when you aren't using mock since 
the path /bin/sh will work on the system you are running on.

[Kevin Fenzi]

On Mon, 9 Sep 2013 22:14:25 -0500
Bruno Wolff III <bruno(a)wolff.to&gt; wrote:

>  On Tue, Sep 10, 2013 at 03:01:08 +0000,
>     Amit Saha <amitksaha(a)fedoraproject.org&gt; wrote:
>  >commit 8f0b1a1f68709622e9b38260ea501bd486f18ac0
>  >Author: Amit Saha <asaha(a)redhat.com&gt;
>  >Date:   Tue Sep 10 13:00:57 2013 +1000
>  >
>  >    Scientific: including sage fails depcheck for no clear reason
>  
>  The error message explains what happened, though it's a bit unusual.
>  sagemath requires /bin/sh and no package appears to provide that. 
>  Either bash should provide both /bin/sh and /usr/bin/sh or sagemath 
>  should require /usr/bin/sh instead of /bin/sh.  
It's not that. ;) 

>  There was some
>  discussion about this problem in general on one of the lists roughly
>  a month ago. Probably the best way to start is by filing a bug with
>  sagemath.
>  
>  Also note, that you won't see this when you aren't using mock since 
>  the path /bin/sh will work on the system you are running on. 
Its actually the pretrans. ;(

You CANNOT use shell in a pretrans. I'm not sure why people keep trying
to do this. Perhaps we could get rpm to throw a build time error... 

Please do file a bug on sagemath. 
https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelin...

kevin

[Amit Saha]

----- Original Message -----
>  From: "Kevin Fenzi" <kevin(a)scrye.com&gt;
>  To: spins(a)lists.fedoraproject.org
>  Sent: Tuesday, September 10, 2013 1:25:26 PM
>  Subject: Re: [Fedora-spins] [spin-kickstarts] Scientific: including sage fails depcheck
> for no clear reason
>  
>  On Mon, 9 Sep 2013 22:14:25 -0500
>  Bruno Wolff III <bruno(a)wolff.to&gt; wrote:
>  
>  > On Tue, Sep 10, 2013 at 03:01:08 +0000,
>  >    Amit Saha <amitksaha(a)fedoraproject.org&gt; wrote:
>  > >commit 8f0b1a1f68709622e9b38260ea501bd486f18ac0
>  > >Author: Amit Saha <asaha(a)redhat.com&gt;
>  > >Date:   Tue Sep 10 13:00:57 2013 +1000
>  > >
>  > >    Scientific: including sage fails depcheck for no clear reason
>  > 
>  > The error message explains what happened, though it's a bit unusual.
>  > sagemath requires /bin/sh and no package appears to provide that.
>  > Either bash should provide both /bin/sh and /usr/bin/sh or sagemath
>  > should require /usr/bin/sh instead of /bin/sh.
>  
>  It's not that. ;)
>  
>  > There was some
>  > discussion about this problem in general on one of the lists roughly
>  > a month ago. Probably the best way to start is by filing a bug with
>  > sagemath.
>  > 
>  > Also note, that you won't see this when you aren't using mock since
>  > the path /bin/sh will work on the system you are running on.
>  
>  Its actually the pretrans. ;(
>  
>  You CANNOT use shell in a pretrans. I'm not sure why people keep trying
>  to do this. Perhaps we could get rpm to throw a build time error...
>  
>  Please do file a bug on sagemath.
>
> https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelin...
Sure, i will do that. Thanks guys.

Best, Amit.

[Amit Saha]

----- Original Message -----
>  From: "Kevin Fenzi" <kevin(a)scrye.com&gt;
>  To: spins(a)lists.fedoraproject.org
>  Sent: Tuesday, September 10, 2013 1:25:26 PM
>  Subject: Re: [Fedora-spins] [spin-kickstarts] Scientific: including sage fails depcheck
> for no clear reason
>  
>  On Mon, 9 Sep 2013 22:14:25 -0500
>  Bruno Wolff III <bruno(a)wolff.to&gt; wrote:
>  
>  > On Tue, Sep 10, 2013 at 03:01:08 +0000,
>  >    Amit Saha <amitksaha(a)fedoraproject.org&gt; wrote:
>  > >commit 8f0b1a1f68709622e9b38260ea501bd486f18ac0
>  > >Author: Amit Saha <asaha(a)redhat.com&gt;
>  > >Date:   Tue Sep 10 13:00:57 2013 +1000
>  > >
>  > >    Scientific: including sage fails depcheck for no clear reason
>  > 
>  > The error message explains what happened, though it's a bit unusual.
>  > sagemath requires /bin/sh and no package appears to provide that.
>  > Either bash should provide both /bin/sh and /usr/bin/sh or sagemath
>  > should require /usr/bin/sh instead of /bin/sh.
>  
>  It's not that. ;)
>  
>  > There was some
>  > discussion about this problem in general on one of the lists roughly
>  > a month ago. Probably the best way to start is by filing a bug with
>  > sagemath.
>  > 
>  > Also note, that you won't see this when you aren't using mock since
>  > the path /bin/sh will work on the system you are running on.
>  
>  Its actually the pretrans. ;(
>  
>  You CANNOT use shell in a pretrans. I'm not sure why people keep trying
>  to do this. Perhaps we could get rpm to throw a build time error... 
You are perhaps referring to this in sage.spec:

########################################################################
%pretrans
# Temporary due to moving directory to symlink
if [ ! -L %{SAGE_ROOT}/devel -a -d %{SAGE_ROOT}/devel ]; then
    mkdir %{SAGE_SRC}
    mv %{SAGE_ROOT}/devel/* %{SAGE_SRC}
    rmdir %{SAGE_ROOT}/devel
    ln -s src %{SAGE_ROOT}/devel
fi


>  
>  Please do file a bug on sagemath. 
Filed: https://bugzilla.redhat.com/show_bug.cgi?id=1006230


Thanks,
Amit.


-- 
Amit Saha <http://echorand.me>
Infrastructure Engineering and Development
Red Hat, Inc.

[Bruno Wolff III]

On Tue, Sep 10, 2013 at 05:35:29 -0400,
   Amit Saha <asaha(a)redhat.com&gt; wrote:
> You are perhaps referring to this in sage.spec:
>
> ########################################################################
> %pretrans
> # Temporary due to moving directory to symlink
> if [ ! -L %{SAGE_ROOT}/devel -a -d %{SAGE_ROOT}/devel ]; then
>     mkdir %{SAGE_SRC}
>     mv %{SAGE_ROOT}/devel/* %{SAGE_SRC}
>     rmdir %{SAGE_ROOT}/devel
>     ln -s src %{SAGE_ROOT}/devel
> fi 
Yes. After reading Kevin's comment I checked and you are limited to using 
Lua scripts in pretrans. (rpm has a builtin lua interpreter.)