On Tue, May 10, 2011 at 09:24:00AM -0500, George McCollister wrote:
>
> >image/env is a better place for this.
> I moved it in image/env like you suggested and changed the name from
> ptx_repo_dist_dir to image_repo_dist_dir. I will retest once other
> changes are ready.
> >if ptxd_get_path "${ptxd_reply%.ipk}.perms"
> >"${ptxd_reply%/*/*}/state/${1}.perms"; then
> > ptxd_reply_perm_files[${#ptxd_reply_perm_files[@]}]="${ptxd_reply}"
> >fi
> >
> >maybe "else ptxd_bailout" if it must exist. I'm actually not sure.
> I don't think it bailed out if the .perms file was missing before, I
> was trying to avoid tinkering with the existing behavior. This seems
> like something that would be appropriate to do as a separate patch
> later if desired. I don't want this alone to hold you from applying
> this patch so if you still want me to do something about this just
> say.
Just submit it without the bailout. I'll add it if I think it's necessary.
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
[email protected]
