[libvirt] spec: Delete .git after applying patches

Message ID 62ff210e5d6542bda3f25ed5e96c5088a840ce1b.1444161965.git.crobinso@redhat.com
State New
Headers show

Commit Message

Cole Robinson Oct. 6, 2015, 8:06 p.m.
I'm hitting this little annoyance in fedora's package repo:

$ fedpkg prep
Downloading libvirt-1.2.20.tar.gz
...
+ /usr/bin/gzip -dc /home/crobinso/src/fedora/libvirt/libvirt-1.2.20.tar.gz
$ git clean -xdf
Removing libvirt-1.2.20.tar.gz
Skipping repository libvirt-1.2.20/

We git-ify the libvirt directory as part of applying patches in the spec
file, but 'git clean' will ignore subfolders that appear to be standalone
git repos.

Let's just delete the .git directory after we're done with it.
---
 libvirt.spec.in | 1 +
 1 file changed, 1 insertion(+)

Comments

Eric Blake Oct. 6, 2015, 8:43 p.m. | #1
On 10/06/2015 02:06 PM, Cole Robinson wrote:
> I'm hitting this little annoyance in fedora's package repo:
> 
> $ fedpkg prep
> Downloading libvirt-1.2.20.tar.gz
> ...
> + /usr/bin/gzip -dc /home/crobinso/src/fedora/libvirt/libvirt-1.2.20.tar.gz
> $ git clean -xdf
> Removing libvirt-1.2.20.tar.gz
> Skipping repository libvirt-1.2.20/
> 
> We git-ify the libvirt directory as part of applying patches in the spec
> file, but 'git clean' will ignore subfolders that appear to be standalone
> git repos.
> 
> Let's just delete the .git directory after we're done with it.
> ---
>  libvirt.spec.in | 1 +
>  1 file changed, 1 insertion(+)

Sounds good to me.  ACK.

Patch hide | download patch | download mbox

diff --git a/libvirt.spec.in b/libvirt.spec.in
index ed401f7..9dff994 100644
--- a/libvirt.spec.in
+++ b/libvirt.spec.in
@@ -1256,6 +1256,7 @@  if [ $COUNT -gt 0 ]; then
 fi
 echo "Applied $COUNT patches"
 rm -f $PATCHLIST
+rm -rf .git
 
 %build
 %if ! %{with_xen}