96386 – subversion.eclass should use svn export when creating ${S}

Bug 96386 - subversion.eclass should use svn export when creating ${S}

Summary: subversion.eclass should use svn export when creating ${S}

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Eclasses (show other bugs)
Hardware:	All All

Importance:	High enhancement (vote)
Assignee:	Akinori Hattori

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2005-06-17 08:23 UTC by REdOG
Modified:	2005-12-24 06:19 UTC (History)
CC List:	1 user (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description REdOG 2005-06-17 08:23:14 UTC

currently the subversion eclass does cp -Rf "${ESVN_STORE_DIR}/${ESVN_CO_DIR}"
"${S}"

This copies the nested .svn repository information that is not necessary for
building the package and wastes time and space.



Reproducible: Always
Steps to Reproduce:
1. use subversion.eclass to create an ebuild that gets its source from a svn
repository
2. emerge it
Actual Results:  
After the potentialy long svn co the working copy is copied to ${S} along with
the nested .svn repository information that is not necessary for building the
package . 
Wastes time and space.

Expected Results:  
(untested)
replaceing
cp -Rf "${ESVN_STORE_DIR}/${ESVN_CO_DIR}" "${S}"
with
svn export "${ESVN_STORE_DIR}/${ESVN_CO_DIR}" "${S}"
should save space and time

I did a quick time test and the results are what I expected 

cp -rf                   |svn export
real    2m58.229s        |real    1m48.375s
user    0m0.428s         |user    0m23.395s
sys     0m4.853s         |sys     0m4.246s
                         |
128M    S                |34M     S

Comment 1 Akinori Hattori gentoo-dev

2005-12-24 06:19:20 UTC

Fixed in CVS. Thank you for reporting!