mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
* [ptxdist] [PATCH] ptxd_lib_dgen.awk: $(STATEDIR)/*.targetinstall.post depends on $(PKGDIR)/Packages
@ 2011-10-27 13:42 Jon Ringle
  2011-10-27 15:12 ` Michael Olbrich
  0 siblings, 1 reply; 5+ messages in thread
From: Jon Ringle @ 2011-10-27 13:42 UTC (permalink / raw)
  To: ptxdist; +Cc: Jon Ringle

From: Jon Ringle <ringlej@jringle-m4600.vertical.com>

Signed-off-by: Jon Ringle <ringlej@jringle-m4600.vertical.com>
---
 scripts/lib/ptxd_lib_dgen.awk |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/scripts/lib/ptxd_lib_dgen.awk b/scripts/lib/ptxd_lib_dgen.awk
index 37ad3d0..a608455 100644
--- a/scripts/lib/ptxd_lib_dgen.awk
+++ b/scripts/lib/ptxd_lib_dgen.awk
@@ -293,6 +293,7 @@ END {
 		if (!(this_pkg ~ /^host-|^cross-/)) {
 			print "$(STATEDIR)/" this_pkg ".targetinstall: "      "$(STATEDIR)/" this_pkg ".install.post"	> DGEN_DEPS_POST;
 			print "$(STATEDIR)/" this_pkg ".targetinstall.post: " "$(STATEDIR)/" this_pkg ".targetinstall"	> DGEN_DEPS_POST;
+			print "$(STATEDIR)/" this_pkg ".targetinstall.post: " "$(PKGDIR)/Packages"              > DGEN_DEPS_POST;
 		}
 
 		#
-- 
1.7.5.4


-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [ptxdist] [PATCH] ptxd_lib_dgen.awk: $(STATEDIR)/*.targetinstall.post depends on $(PKGDIR)/Packages
  2011-10-27 13:42 [ptxdist] [PATCH] ptxd_lib_dgen.awk: $(STATEDIR)/*.targetinstall.post depends on $(PKGDIR)/Packages Jon Ringle
@ 2011-10-27 15:12 ` Michael Olbrich
  2011-10-27 16:02   ` Jon Ringle
  0 siblings, 1 reply; 5+ messages in thread
From: Michael Olbrich @ 2011-10-27 15:12 UTC (permalink / raw)
  To: ptxdist

Please elaborate why this is necessary.

Michael

On Thu, Oct 27, 2011 at 09:42:31AM -0400, Jon Ringle wrote:
> From: Jon Ringle <ringlej@jringle-m4600.vertical.com>
> 
> Signed-off-by: Jon Ringle <ringlej@jringle-m4600.vertical.com>
> ---
>  scripts/lib/ptxd_lib_dgen.awk |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/scripts/lib/ptxd_lib_dgen.awk b/scripts/lib/ptxd_lib_dgen.awk
> index 37ad3d0..a608455 100644
> --- a/scripts/lib/ptxd_lib_dgen.awk
> +++ b/scripts/lib/ptxd_lib_dgen.awk
> @@ -293,6 +293,7 @@ END {
>  		if (!(this_pkg ~ /^host-|^cross-/)) {
>  			print "$(STATEDIR)/" this_pkg ".targetinstall: "      "$(STATEDIR)/" this_pkg ".install.post"	> DGEN_DEPS_POST;
>  			print "$(STATEDIR)/" this_pkg ".targetinstall.post: " "$(STATEDIR)/" this_pkg ".targetinstall"	> DGEN_DEPS_POST;
> +			print "$(STATEDIR)/" this_pkg ".targetinstall.post: " "$(PKGDIR)/Packages"              > DGEN_DEPS_POST;
>  		}
>  
>  		#
> -- 
> 1.7.5.4
> 
> 
> -- 
> ptxdist mailing list
> ptxdist@pengutronix.de
> 

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [ptxdist] [PATCH] ptxd_lib_dgen.awk: $(STATEDIR)/*.targetinstall.post depends on $(PKGDIR)/Packages
  2011-10-27 15:12 ` Michael Olbrich
@ 2011-10-27 16:02   ` Jon Ringle
  2011-10-27 17:05     ` Michael Olbrich
  0 siblings, 1 reply; 5+ messages in thread
From: Jon Ringle @ 2011-10-27 16:02 UTC (permalink / raw)
  To: ptxdist

On Thu, Oct 27, 2011 at 11:12 AM, Michael Olbrich
<m.olbrich@pengutronix.de> wrote:
> Please elaborate why this is necessary.

Doing this makes it possible for a developer to point opkg.conf on the
target to a URL that resolves to platform/packages/. A developer that
is working on a single package can then just do ptxdist targetinstall
foo and on the target:
opkg update
opkg remove foo
opkg install foo

Doing ptxdist images and having the ipkg-repository updated takes too
long when all one is interested is the update of a single .ipkg file

Jon

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [ptxdist] [PATCH] ptxd_lib_dgen.awk: $(STATEDIR)/*.targetinstall.post depends on $(PKGDIR)/Packages
  2011-10-27 16:02   ` Jon Ringle
@ 2011-10-27 17:05     ` Michael Olbrich
  2011-10-27 17:21       ` Jon Ringle
  0 siblings, 1 reply; 5+ messages in thread
From: Michael Olbrich @ 2011-10-27 17:05 UTC (permalink / raw)
  To: ptxdist

On Thu, Oct 27, 2011 at 12:02:21PM -0400, Jon Ringle wrote:
> On Thu, Oct 27, 2011 at 11:12 AM, Michael Olbrich
> <m.olbrich@pengutronix.de> wrote:
> > Please elaborate why this is necessary.
> 
> Doing this makes it possible for a developer to point opkg.conf on the
> target to a URL that resolves to platform/packages/. A developer that
> is working on a single package can then just do ptxdist targetinstall
> foo and on the target:
> opkg update
> opkg remove foo
> opkg install foo
> 
> Doing ptxdist images and having the ipkg-repository updated takes too
> long when all one is interested is the update of a single .ipkg file

I see what you mean. However I'm pretty sure, that with this patch if you
run "ptxdist images" in a clean BSP you'll get a broken Packages file
because it's created too early.
It running "ptxdist targetinstall foo && ptxdist make ipkg-index" not good
enough?

Michael

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [ptxdist] [PATCH] ptxd_lib_dgen.awk: $(STATEDIR)/*.targetinstall.post depends on $(PKGDIR)/Packages
  2011-10-27 17:05     ` Michael Olbrich
@ 2011-10-27 17:21       ` Jon Ringle
  0 siblings, 0 replies; 5+ messages in thread
From: Jon Ringle @ 2011-10-27 17:21 UTC (permalink / raw)
  To: ptxdist

On Thu, Oct 27, 2011 at 1:05 PM, Michael Olbrich
<m.olbrich@pengutronix.de> wrote:
> On Thu, Oct 27, 2011 at 12:02:21PM -0400, Jon Ringle wrote:
>> On Thu, Oct 27, 2011 at 11:12 AM, Michael Olbrich
>> <m.olbrich@pengutronix.de> wrote:
>> > Please elaborate why this is necessary.
>>
>> Doing this makes it possible for a developer to point opkg.conf on the
>> target to a URL that resolves to platform/packages/. A developer that
>> is working on a single package can then just do ptxdist targetinstall
>> foo and on the target:
>> opkg update
>> opkg remove foo
>> opkg install foo
>>
>> Doing ptxdist images and having the ipkg-repository updated takes too
>> long when all one is interested is the update of a single .ipkg file
>
> I see what you mean. However I'm pretty sure, that with this patch if you
> run "ptxdist images" in a clean BSP you'll get a broken Packages file
> because it's created too early.
> It running "ptxdist targetinstall foo && ptxdist make ipkg-index" not good
> enough?

I'm sure that "ptxdist make ipkg-index" will work. I just didn't know
how to invoke that rule before now :)

Jon

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2011-10-27 17:21 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-10-27 13:42 [ptxdist] [PATCH] ptxd_lib_dgen.awk: $(STATEDIR)/*.targetinstall.post depends on $(PKGDIR)/Packages Jon Ringle
2011-10-27 15:12 ` Michael Olbrich
2011-10-27 16:02   ` Jon Ringle
2011-10-27 17:05     ` Michael Olbrich
2011-10-27 17:21       ` Jon Ringle

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox