~alpine/devel

This thread contains a patchset. You're looking at the original emails, but you may wish to use the patch review UI. Review patch
3 2

[alpine-devel] [PATCH 0/2] two rebased apk-tools patches

Details
Message ID
<cover.1372579185.git.dubiousjim@gmail.com>
Sender timestamp
1372579337
DKIM signature
missing
Download raw message
On Fri, Jun 28, 2013 at 02:08:09PM +0300, Timo Teras wrote:
> On Thu, 27 Jun 2013 15:28:52 -0400
> Dubiousjim <dubiousjim@gmail.com> wrote:
> 
> > Also add infrastructure for short-option-only synonyms
> 
> Care to split the infrastructure to separate commit?
> 
> I'm willing to commit that, along with the "search -x" alias and
> reordering of the options where needed, before tagging 2.4.0.
> 
> The -R / -D options thingy, along with version, index, search and info
> applet changes will be postponed.
> 
> I pushed the first four commits already with ncopa's Ack. I did minor
> change as suggested by ncopa to one of the commits, and reworded the
> commit message.
> 
> Please reword the commit messages similarly if resending the above
> mentioned commits.


Here are the requested two patches, rebased onto the current master.


Dubiousjim (2):
  Allow "apk search -x" for -e
  apk: infrastructure for short-option-only synonyms

 src/apk.c    | 17 +++++++++--------
 src/search.c |  4 +++-
 2 files changed, 12 insertions(+), 9 deletions(-)

-- 
1.8.3.1



---
Unsubscribe:  alpine-devel+unsubscribe@lists.alpinelinux.org
Help:         alpine-devel+help@lists.alpinelinux.org
---

[alpine-devel] [PATCH 1/2] Allow "apk search -x" for -e

Details
Message ID
<c92b86ca625d2b45914e86047e34b2a3266be591.1372579185.git.dubiousjim@gmail.com>
In-Reply-To
<cover.1372579185.git.dubiousjim@gmail.com> (view parent)
Sender timestamp
1372579338
DKIM signature
missing
Download raw message
Patch: +3 -1
---
 src/search.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/search.c b/src/search.c
index 7ccf98a..c85892b 100644
--- a/src/search.c
+++ b/src/search.c
@@ -79,6 +79,7 @@ static int search_parse(void *ctx, struct apk_db_options *dbopts,
		ictx->show_all = 1;
		break;
	case 'e':
	case 'x':
		ictx->search_exact = 1;
		break;
	case 'o':
@@ -166,7 +167,8 @@ static int search_main(void *pctx, struct apk_database *db, struct apk_string_ar
static struct apk_option search_options[] = {
	{ 'a', "all",		"Show all package versions (instead of latest only)" },
	{ 'd', "description",	"Search package descriptions (implies -a)" },
	{ 'e', "exact",		"Require exact match (instead of substring match)" },
	{ 'x', "exact",		"Require exact match (instead of substring match)" },
	{ 'e', NULL,	        "Synonym for -x (deprecated)" },
	{ 'o', "origin",	"Print origin package name instead of the subpackage" },
	{ 'r', "rdepends",	"Print reverse dependencies of package" },
};
-- 
1.8.3.1



---
Unsubscribe:  alpine-devel+unsubscribe@lists.alpinelinux.org
Help:         alpine-devel+help@lists.alpinelinux.org
---

[alpine-devel] [PATCH 2/2] apk: infrastructure for short-option-only synonyms

Details
Message ID
<6e2248ca48687bc77837f1ef6fb8784f8c0a70ff.1372579185.git.dubiousjim@gmail.com>
In-Reply-To
<cover.1372579185.git.dubiousjim@gmail.com> (view parent)
Sender timestamp
1372579339
DKIM signature
missing
Download raw message
Patch: +9 -8
---
 src/apk.c | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/src/apk.c b/src/apk.c
index 17e87d5..a236b37 100644
--- a/src/apk.c
+++ b/src/apk.c
@@ -111,13 +111,14 @@ static void print_usage(const char *cmd, const char *args, int num_opts,
	int i, j;

	indent.x = printf("\nusage: apk %s", cmd) - 1;
	for (i = 0; i < num_opts; i++) {
		j = 0;
		word[j++] = '[';
		j += format_option(&word[j], sizeof(word) - j, &opts[i], "|");
		word[j++] = ']';
		apk_print_indented(&indent, APK_BLOB_PTR_LEN(word, j));
	}
	for (i = 0; i < num_opts; i++)
		if (opts[i].name != NULL) {
			j = 0;
			word[j++] = '[';
			j += format_option(&word[j], sizeof(word) - j, &opts[i], "|");
			word[j++] = ']';
			apk_print_indented(&indent, APK_BLOB_PTR_LEN(word, j));
		}
	if (args != NULL)
		apk_print_indented(&indent, APK_BLOB_STR(args));
	printf("\n");
@@ -230,7 +231,7 @@ static void merge_options(struct option *opts, struct apk_option *ao, int num)
	int i;

	for (i = 0; i < num; i++, opts++, ao++) {
		opts->name = ao->name;
		opts->name = ao->name ?: "";
		opts->has_arg = ao->has_arg;
		opts->flag = NULL;
		opts->val = ao->val;
-- 
1.8.3.1



---
Unsubscribe:  alpine-devel+unsubscribe@lists.alpinelinux.org
Help:         alpine-devel+help@lists.alpinelinux.org
---
Timo Teras <timo.teras@iki.fi>
Details
Message ID
<20130630111945.0f02619e@vostro>
In-Reply-To
<cover.1372579185.git.dubiousjim@gmail.com> (view parent)
Sender timestamp
1372580385
DKIM signature
missing
Download raw message
On Sun, 30 Jun 2013 04:02:17 -0400
Dubiousjim <dubiousjim@gmail.com> wrote:

> Here are the requested two patches, rebased onto the current master.
> 
> Dubiousjim (2):
>   Allow "apk search -x" for -e
>   apk: infrastructure for short-option-only synonyms
> 
>  src/apk.c    | 17 +++++++++--------
>  src/search.c |  4 +++-
>  2 files changed, 12 insertions(+), 9 deletions(-)

Pushed in reverse order as the dependency goes that way.

Thanks,
 Timo


---
Unsubscribe:  alpine-devel+unsubscribe@lists.alpinelinux.org
Help:         alpine-devel+help@lists.alpinelinux.org
---
Reply to thread Export thread (mbox)