[Debian-science-sagemath] Fwd: sagemath_7.4-1_amd64.changes is NEW [sympow/1.023-8]

[Jerome BENOIT]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Hello Forum

On 18/12/16 15:18, Jerome BENOIT wrote:
> Hello, thanks for the precision.
> 
> On 18/12/16 10:09, Ximin Luo wrote:
>> Control: retitle -1 Document SYMPOW_CACHEDIR better and automatically try to create it
> 
>> Jerome BENOIT:
>>> Hello SymPow enthusiasts,
>>>
>>> On 17/12/16 15:46, Jerome BENOIT wrote:
>>>>> It would be good if sympow could automatically create SYMPOW_CACHEDIR
>>>>>> if it doesn't exist. Otherwise, I would have to add this logic to
>>>>>> Sage, which seems a bit unclean.
>>>> I not so sure that it is a good idea to create SYMPOW_CACHEDIR on the fly.
>>>> Not that if SYMPOW_CACHEDIR does not exist, an appropraite message is printed.
>>>
>>>>>
>>> I am on my way to add a mechanism to create SYMPOW_CACHEDIR on the fly
>>> (that may avoid to add a dirty trick on Sage).
>>>
> 
>> Hey, thanks for this. To explain in some more detail: at the moment I have to patch Sage like this:
> 
>> sage/src/sage/lfunctions/sympow.py:
>> -        cmd = 'sympow %s'%args
>> +        cmd = 'env HOME="%s" sympow %s' % (DOT_SAGE, args)
> 
>> Setting HOME is ugly, it would be nicer to do this instead:
> 
>> -        cmd = 'sympow %s'%args
>> +        cmd = 'env SYMPOW_CACHEDIR="%s" sympow %s' % (os.path.join(DOT_SAGE, "sympow"), args)
> 
>> However I can't, because sympow does not automatically create SYMPOW_CACHEDIR. I would have to do this instead:
> 
>> -        cmd = 'sympow %s'%args
>> +        cmd = 'mkdir "{0}" && env SYMPOW_CACHEDIR="{0}" sympow {1}'.format(os.path.join(DOT_SAGE, "sympow"), args)
> 
>> Note, it's not necessary to create all the parent directories like what `mkdir -p` does. That is, if `dirname $SYMPOW_CACHEDIR` does not exist, I think it's fine that sympow fails - because there is no way to know what permissions (etc) to create parent directories with, it is a sysadmin and local policy issue, and it means there is a bug somewhere else.
> 
>> However if `dirname $SYMPOW_CACHEDIR` does exist, then sympow should try to create SYMPOW_CACHEDIR inside it - it "controls" this directory, so it knows what permissions to use.
> 
>> So actually, I think there is no bug in sympow regarding non-existent HOME. It is OK to fail, if HOME does not exist and SYMPOW_CACHEDIR is not set. It is better that during tests, SYMPOW_CACHEDIR is set explicitly in a Makefile, or by the parent process (Sage) which already has a "directory for testing" (DOT_SAGE).
> 
> 
> I am working on it. It would be something like:
> 
> cmd = 'env SYMPOW_CACHEDIR="%s///" sympow %s' % (os.path.join(DOT_SAGE, "sympow"), args)
> 
> (Note the magic triplet `///').
> Please give one more night: I want to implement something consistent and solid.

I needed more time, but I did.
The new version is currently on its way to Sid.
Note that a README.Debian has been introduced.

> 
> Nevertheless, if it is a just building issue/feature, have you consider to just MKDIR_P in in d/rules ?
> 
> For the recoard, the issue is not bug but sbuild feature:
> 
> https://lists.debian.org/debian-devel/2016/02/msg00454.html
> 
> 
> Thanks,
> Jerome

Thanks,
Jerome

> 
> 
>> X
> 
> 
> 
> _______________________________________________
> Debian-science-sagemath mailing list
> Debian-science-sagemath at lists.alioth.debian.org
> https://lists.alioth.debian.org/mailman/listinfo/debian-science-sagemath
> 

- -- 
Jerome BENOIT, Ph.D. | jgmbenoit-at+rezozer*dot_net
http://www.rezozer.net/

- -- 
Jerome BENOIT | calculus+at-rezozer^dot*net
https://qa.debian.org/developer.php?login=calculus@rezozer.net
AE28 AE15 710D FF1D 87E5  A762 3F92 19A6 7F36 C68B
-----BEGIN PGP SIGNATURE-----

iQQcBAEBCgAGBQJYYUc0AAoJED+SGaZ/NsaL4PogAINPYTR+uGTKqf6cnQlKFyr4
5GNBT3iw+VFn0kWpkVBySn8b0ey3bj93RBh9293AzXL2LMuXhZ6DqyqUgCGFI/+v
SAlOR5eWNe8YV92tiB8+znYMoi3TNH+KQaECHeqxbd4xkX3uGKmU37PXneW4Lw9p
PYxQs5tJIiDpEwek+gPGSfe4nyIv0t4yTwV7fnoQCzTZY5+S5cD4pB7jHTc7ZIVT
LbQHj+/VXyMsnTMyB0/eDqPltBvFrJq5yE01E3eOdGZvZB8BZk8C21mRT0+4/edS
auGgFwj5pVDiwo5pOuVLVcSAZdu/MkCnW3UKYLQTUB5BTR8hFB7kJrbnGVrW37DH
9StL7hcoj1EGI2TfbACNI9VvUjWyaHViaswV79OiLI2ylhxY28qs+xVW8D5Y+5UY
STGdNAb+Bhu+DEYfq1DeX3mtzWFNqvAvV5Kaa58f10hib8zcEfJwmk6em+vQJ0gI
cF6unLKQ7plOodIAhFAkIvViVKQf/rxozLbVAmvndlPpXeg7DdzsE6oc6Ay7K0pv
XBQMtWyUP+AzM7hyKplrjUxyVEWjDRaOPR9P+UQrp9PxcYJ3oB0aW5i/u4oSzy6r
J22jWVyFFa9TspIgD+w2MjFt0m0GBja7GwrGyRBOz0ers5LFsLXuY04amBe5DIEa
yYHw0NNP8QiXYsOLKwizx6yBp3TQeRIEUIwkN33Rx+wxEL44nSNOazaL5714NIfi
9FmzeTaMRTOBOEsE9UPuBPuoGOXbqdmaI2McO0mXsymQblDUXH18kvthaQAvDrYI
ehGiN2gHXqr49beNEpKboUGnfNwqoyuIAAmIi+T+vkcO0Nz0DN1YybBz4gA+u9WJ
EGj9Gh/EKszKtmwuDn4sJgV69vZ7EwIHNybkGzkork1oKY8tvX9LZkX+SGYi5/k2
bU+lKrNmHwO8uG50gy4HGw44Ov65IvRf1gthUgUDfmo9d2kb3ujOk7aiAKprsSMW
xUwqron3iWVqpYfzX4Uzh1nYe8wBA9TiIHL0cPO0qaNW24umccW1MBafT4g2vpwL
Yo0kG0jgLdTjtJdkW32AOVeNwxIZ+1DG3sPn5a/uShQBwLqAHWTQM63bcBEnbQkb
dSukdoIW4fKQWFIsP/axsKyozWZ5/opYIAOfuBQu/dyn/sGDtDcz+b5LiN6AYhVC
YK0zD8j4EZNikidRKerBMq1nNrSc0TfCokQT07vVDUIp09VNkNcgJkhU6/y3BDB1
MPfbx8NAB5ugixa0GHwJigsW3n01Q1iCT3zc9jiEj0NVUqfC96m/Lpgu+QanL8t4
JXx6bp9AeQv2e9lIYN73TRCvvhSBqr5DDSdNZkotmc4uiwRbtDMwCNrxsKGL3x0=
=eAmT
-----END PGP SIGNATURE-----