First cut a new mock launcher

Clark Williams williams at redhat.com
Wed Jun 14 18:34:41 UTC 2006


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

Michael_E_Brown at Dell.com wrote:
> Nice start. A couple discussion points:
>
> void debug(const char *format, ...)
> {
> #ifdef DEBUG
>
> instead of #ifdef DEBUG, can we have an env var that controls this
> on/off? (No recompile)

Thought about that. I didn't know if we wanted a "production" version
that didn't have all the debug spew. I can rework it to use an
environment variable.

>
> #define NSFLAG        "NAMESPACE=1"
>
> What is the purpose of this?
>

I was unsure if we needed to know that we were running with namespaces
and modify our behavior in mock.py. If it turns out we don't need to
do anything differently, I'll pull this out of the environment.

>
>     // elevate our privileges
>     if (setreuid (geteuid (), geteuid ()))
>         error("setreuid failed: %s", strerror(errno));
>     debug("running with uid: %d, gid: %d\n", getuid(), getgid());
>
> Is this necessary? It seems to me like this is something that can be
> done in mock.py for the cases where it is necessary, as mock.py might
> need to know the user id of the person running mock, and so it can
> switch back to that user for writing output files.

So, you're thinking we just to seteuid/setegid rather than setting the
real uid/gid?

Clark

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.3 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org

iD8DBQFEkFbAHyuj/+TTEp0RArRDAKCtiesHKyKDDraA4wHr9T6IkU8xbQCfdmRi
GikSVUVRgLWZVRbI87CNf7I=
=cQbq
-----END PGP SIGNATURE-----




More information about the Fedora-buildsys-list mailing list