FS#59102 - [tensorflow] v1.8.0-3 incorrect path in PKGBUILD for python-tensorflow-opt-cuda split package

Attached to Project: Community Packages
Opened by David Palma (palma) - Thursday, 21 June 2018, 17:05 GMT
Last edited by Sven-Hendrik Haase (Svenstaro) - Tuesday, 03 July 2018, 20:29 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Sven-Hendrik Haase (Svenstaro)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

When going through the PKGBUILD of tensorflow-1.8.0-3, for creating a local build, I realised that the packaging of `python-tensorflow-opt-cuda` is pointing to the incorrect source folder.

I believe the following patch would correct this:

```
--- PKGBUILD.orig 2018-06-21 20:36:47.660858518 +0200
+++ PKGBUILD.new 2018-06-21 20:37:44.900859413 +0200
@@ -205,7 +205,7 @@
provides=(python-tensorflow python-tensorflow-cuda)
pkgdesc="Library for computation using data flow graphs for scalable machine learning (with CUDA and CPU optimizations)"

- cd ${srcdir}/tensorflow-${_pkgver}-cuda
+ cd ${srcdir}/tensorflow-${_pkgver}-opt-cuda

WHEEL_PACKAGE=$(find ${srcdir}/tmpcudaopt -name "tensor*.whl")
pip install --ignore-installed --upgrade --root $pkgdir/ $WHEEL_PACKAGE --no-dependencies
```

Additional info:
% pacman -Qi tensorflow
Name : tensorflow-opt-cuda
Version : 1.8.0-3
Description : Library for computation using data flow graphs for scalable machine learning (with CUDA and CPU optimizations)
Architecture : x86_64
URL : https://www.tensorflow.org/
Licenses : APACHE
Groups : None
Provides : tensorflow tensorflow-cuda
Depends On : cuda cudnn
Optional Deps : tensorboard: Tensorflow visualization toolkit [installed]
Required By : None
Optional For : None
Conflicts With : tensorflow
Replaces : None
Installed Size : 609.73 MiB
Packager : Sven-Hendrik Haase <svenstaro@gmail.com>
Build Date : Mon 04 Jun 2018 20:10:45 CEST
Install Date : Thu 21 Jun 2018 18:41:48 CEST
Install Reason : Explicitly installed
Install Script : No
Validated By : Signature


Steps to reproduce:
1. asp checkout tensorflow
2. cat tensorflow/trunk/PKGBUILD
This task depends upon

Closed by  Sven-Hendrik Haase (Svenstaro)
Tuesday, 03 July 2018, 20:29 GMT
Reason for closing:  Fixed
Comment by Sven-Hendrik Haase (Svenstaro) - Monday, 25 June 2018, 17:40 GMT
Try now.
Comment by David Palma (palma) - Tuesday, 26 June 2018, 06:34 GMT
Thanks, the online version looks fine to me (for some reason `git pull` or even `asp checkout` are not updating the files...).

Loading...