1 |
On Sat, 27 Jul 2002 16:05:58 +0200 |
2 |
Marko Mikulicic <marko@××××.org> wrote: |
3 |
|
4 |
> mikepolniak wrote: |
5 |
> > On Fri, 26 Jul 2002 22:50:41 +0200 |
6 |
> > Marko Mikulicic <marko@××××.org> wrote: |
7 |
> > |
8 |
> > |
9 |
> |
10 |
> > |
11 |
> > Since /usr/portage/incoming/apmd/ does not have an ebuild file the script |
12 |
> > fails with: |
13 |
> > |
14 |
> > ls: /usr/portage/incoming/apmd/*.ebuild: No such file or directory |
15 |
> > ./nuebuild: .: filename argument required |
16 |
> |
17 |
> the script should go on. |
18 |
> It will report the error but it should go on. |
19 |
> I know it's annoying, but I don't know if its worth |
20 |
> complicating the script just for handling a exceptional condition |
21 |
> which should not be there. (If the program doesn't abort) |
22 |
> However I have the patch: |
23 |
> |
24 |
> <snip> |
25 |
> ** /usr/local/bin/emerge-rsync 2002-07-26 22:47:41.000000000 +0200 |
26 |
> --- emerge-rsync 2002-07-27 16:00:39.000000000 +0200 |
27 |
> *************** |
28 |
> *** 34,40 **** |
29 |
> echo |
30 |
> echo New ebuilds: |
31 |
> for i in $(cat $NEW); do |
32 |
> ! . $(ls $i/*.ebuild --sort=time | head -n 1) |
33 |
> echo -e $GREEN${i##$PORTDIR/}$NORMAL: $DESCRIPTION |
34 |
> done |
35 |
> fi |
36 |
> --- 34,42 ---- |
37 |
> echo |
38 |
> echo New ebuilds: |
39 |
> for i in $(cat $NEW); do |
40 |
> ! DESCRIPTION="" |
41 |
> ! EBUILD=$(ls $i/*.ebuild --sort=time 2>/dev/null | head -n 1) |
42 |
> ! [ -z "$EBUILD" ] || . $EBUILD |
43 |
> echo -e $GREEN${i##$PORTDIR/}$NORMAL: $DESCRIPTION |
44 |
> done |
45 |
> fi |
46 |
> </snip> |
47 |
> |
48 |
> But I would like to hear opinions. |
49 |
> Is it normal to have directories with no *.emerge? |
50 |
> Should it report an error message or simply discard the description |
51 |
> (as the patch does) ? |
52 |
> Clean is nice, but real world is ugly :-) |
53 |
> |
54 |
I did echo "$i/*.ebuild does not exist" for the error message. |
55 |
|
56 |
I also like your idea to EXCLUDE paths since e.g. i dont use gnome or kde and |
57 |
would exclude their listing. |