1 |
For normal users we wouldn't. But currently, arch-testers need to make a |
2 |
judgement call on what to test when a stable-req bug is filed. Tests run in |
3 |
src_test are provided by upstream, and does not guarantee that a package |
4 |
that has been merged will actually run on the system. |
5 |
If the maintainer could add a couple small scripts to check basic |
6 |
functionality |
7 |
of the merged package, it would make testing for arch testers much easier |
8 |
and reliable. |
9 |
Let me give an example. Let's say |
10 |
app-misc/screenfetch-2.7.7 is the current stable package for screenfetch in |
11 |
the portage tree. |
12 |
However, on running, it produces an error on the top, along with the proper |
13 |
output. |
14 |
If screenfetch-3.0.0 happens to fix that error, maintainer can add a simple |
15 |
script |
16 |
|
17 |
if [ "$(screenfetch 2>&1 1>/dev/null)" != "" ] then |
18 |
eerror "Still producing error" |
19 |
fi |
20 |
|
21 |
To make sure the build is properly updating the screenfetch version, and |
22 |
that |
23 |
the bug has in fact been fixed. This is the only way I can see to reliabily |
24 |
and automatically test packages that have been merged successfully. |
25 |
|
26 |
------- |
27 |
Regards, |
28 |
Pallav |
29 |
|
30 |
On Mon, May 16, 2016 at 10:08 PM, Luis Ressel <aranea@×××××.de> wrote: |
31 |
|
32 |
> On Mon, 16 May 2016 18:13:33 +0530 |
33 |
> Pallav Agarwal <pallavagarwal07@×××××.com> wrote: |
34 |
> |
35 |
> > What I'm suggesting is to add a new function post_install_test. The |
36 |
> > function will run only if the build is being run for stabilization |
37 |
> > (either as a part of automated stabilization, or manual) which can be |
38 |
> > controlled by a USE flag. The function would also require independent |
39 |
> > dependencies in case it uses external applications to test the one |
40 |
> > being built. |
41 |
> |
42 |
> Could you please elaborate on this? We already have src_test() for |
43 |
> automated tests. Why would we want to run additional tests after the |
44 |
> package has been merged? |
45 |
> |
46 |
> -- |
47 |
> Regards, |
48 |
> Luis Ressel |
49 |
> |
50 |
> Luis Ressel <aranea@×××××.de> |
51 |
> GPG fpr: F08D 2AF6 655E 25DE 52BC E53D 08F5 7F90 3029 B5BD |
52 |
> |