[pacman-dev] [PATCH v3 1/2] makepkg: Move parseopts from library to libmakepkg

Eli Schwartz eschwartz93 at gmail.com
Sun Oct 9 01:04:17 UTC 2016


On 10/08/2016 01:34 PM, Alad Wenter wrote:
> parseopts is used in makepkg and other scripts such as pacman-key as a
> getopt replacement.
> 
> Instead of including it in those scripts via a macro, move it to
> libmakepkg/util/option.sh and have scripts source libmakepkg where
> appropriate.

Is it necessary for all those scripts to source the entirety of
libmakepkg? In fact, many parts of libmakepkg source a specific file
that that libmakepkg component needs.

I also don't think it should go in util/option.sh which doesn't have
anything to do with getopt/parseopts/any sort of command-line opts at all.
The header states:

```
option.sh - functions to test if build/packaging options are enabled
```

Why not move it to util/parseopts.sh if it should be moved into libmakepkg?

-- 
Eli Schwartz


More information about the pacman-dev mailing list