[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/5] exec: remove support for transparently unbzip2ing execut
From: |
Samuel Thibault |
Subject: |
Re: [PATCH 1/5] exec: remove support for transparently unbzip2ing executables |
Date: |
Thu, 29 Aug 2013 12:32:50 +0200 |
User-agent: |
Mutt/1.5.21+34 (58baf7c9f32f) (2010-12-30) |
Justus Winter, le Thu 29 Aug 2013 12:16:11 +0200, a écrit :
> The patches were also ment to address the complexity^Wsheer size of
> the code. That is why I wanted to remove them, #ifdef'ing it out has a
> cost, not a runtime one but for anyone hacking on /hurd/exec.
Yes, the BFD code was making the whole exec code a lot more complex to
understand. Since even the TODO talks about ditching it away, and
reviving and maintaining it would be a burden, it's better to just not
keep it, and have readable exec instead.
> Out of curiosity, why do you consider this feature interesting?
At least to show flexibility of the exec server. The difference between
the BFD code and the gzip/bzip2 code is that the latter makes the whole
exec code complex, while the gzip/bzip2 support only has a couple of
hooks, so even if the feature doesn't seem so appealing, the maintenance
cost is light.
Samuel
- Cleanups for the exec server, Justus Winter, 2013/08/15
- [PATCH 1/5] exec: remove support for transparently unbzip2ing executables, Justus Winter, 2013/08/15
- Re: [PATCH 1/5] exec: remove support for transparently unbzip2ing executables, Samuel Thibault, 2013/08/28
- Re: [PATCH 1/5] exec: remove support for transparently unbzip2ing executables, Justus Winter, 2013/08/29
- Re: [PATCH 1/5] exec: remove support for transparently unbzip2ing executables,
Samuel Thibault <=
- Re: [PATCH 1/5] exec: remove support for transparently unbzip2ing executables, Justus Winter, 2013/08/29
- Re: [PATCH 1/5] exec: remove support for transparently unbzip2ing executables, Samuel Thibault, 2013/08/29
- Re: [PATCH 1/5] exec: remove support for transparently unbzip2ing executables, Justus Winter, 2013/08/29
- Re: [PATCH 1/5] exec: remove support for transparently unbzip2ing executables, Samuel Thibault, 2013/08/29
- Re: [PATCH 1/5] exec: remove support for transparently unbzip2ing executables, Richard Braun, 2013/08/29
- Re: [PATCH 1/5] exec: remove support for transparently unbzip2ing executables, Roland McGrath, 2013/08/29
[PATCH 3/5] exec: Remove #ifdef 0-out code doing nothing., Justus Winter, 2013/08/15
[PATCH 5/5] exec: remove the BFD code, Justus Winter, 2013/08/15