Re: fedpkg prep

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 8/31/10 3:48 PM, Steve Dickson wrote:
> Just curious....
> 
> Why does 'fedpkg prep' care that the repo is in an inconsistent state?
> 
> I just did a rebase to the latest f14 code on a private branch. 
> So yes he repo in an inconsistent but that is ok. I'm going to
> make some changes to put it back in a consistent state but 
> I can not do that without a source tree.
> 
> So is there a way to do what the old 'make prep' did? That is
> stupidity untar the tarball an apply all the current patch 
> without checking any state??
> 
> tia,
> 
> steved.
>  

It only cares in that the "prep" function is a function of the
PackageModule class.  When we create an object of the PackageModule
class, we also create an object for the git repo.  It's when trying to
create the object for the git repo that git is returning us an error,
which we are passing along.  It doesn't have anything to do with "prep"
itself, any action which would necessitate creating the PackageModule
object would run into this issue.

Couple things I've been thinking about.  We could delay creating the git
object unless calling a function that requires it.  We could move some
of the rpm commands out of the PacakgeModule class.  We could catch this
and not care unless we are doing a git action.

Either way, in this case your git repo does seem kinda broken, and you
might want to fix that.

- -- 
Jesse Keating
Fedora -- Freedom² is a feature!
identi.ca: http://identi.ca/jkeating


-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Darwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAkx9lAoACgkQ4v2HLvE71NWUngCgvSQ6uKm1Zztq8Vfc4ZcYjiSo
bkgAn2SbhO4ppPMfttp3+y0JhXzyaHT8
=N/Xk
-----END PGP SIGNATURE-----
-- 
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/devel



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]
  Powered by Linux