Review Request: Improvements to create_tarball so it can properly package kio_gopher
Albert Astals Cid
aacid at kde.org
Sat Dec 17 17:00:25 GMT 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/6864/
-----------------------------------------------------------
Review request for kdelibs, Release Team and Kåre Särs.
Description
-------
kio_gopher is a kioslave, thus it's docs are not in docs/extragear-network/kio_gopher but in docs/extragear-network/kioslave/gopher/
I addded a few special casing for when an addDocs entry contains a / since you need recursive mkdir and some special casing in the copy.
I could probably fold the old code with the new one, but did not want to break anything and since i ruby knowledge is totally non existent i decided to go the safe way.
Kare, i'm also adding you because i'm removing a
`mkdir #{app}`
you added that i don't see why it's needed, it was basically creating an empty dir in my src dir.
Diffs
-----
home/kde/trunk/KDE/kdesdk/scripts/createtarball/create_tarball.rb 1268943
Diff: http://svn.reviewboard.kde.org/r/6864/diff/diff
Testing
-------
I now can build kio_gopher with the following config
[kio-gopher]
gitModule = yes
mainmodule = extragear
submodule = network
kde_release = no
docs = yes
addDocs = kioslave/gopher
translations = yes
version = 0.1.4
custompo = kio_gopher
Thanks,
Albert Astals Cid
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20111217/e122ae15/attachment.htm>
More information about the kde-core-devel
mailing list