[elbe-devel] [PATCH 01/13] examples/beaglebone: add security and backports

Manuel Traut manuel.traut at linutronix.de
Fri Dec 8 17:34:08 CET 2017


On Tue, Dec 05, 2017 at 03:50:56PM +0100, Torben Hohn wrote:
> On Thu, Nov 30, 2017 at 03:15:03PM +0100, Manuel Traut wrote:
> > adds security and backports debian repos.
> > 
> > To show the usage of pinnig to a repo, btrfs-progs are installed from
> > backports.
> > 
> > Signed-off-by: Manuel Traut <manut at linutronix.de>
> > ---
> >  examples/armhf-ti-beaglebone-black.xml | 14 ++++++++++++++
> >  1 file changed, 14 insertions(+)
> > 
> > diff --git a/examples/armhf-ti-beaglebone-black.xml b/examples/armhf-ti-beaglebone-black.xml
> > index 3dcf130d..f0c68a25 100644
> > --- a/examples/armhf-ti-beaglebone-black.xml
> > +++ b/examples/armhf-ti-beaglebone-black.xml
> > @@ -13,7 +13,19 @@
> >  			<primary_host>ftp.de.debian.org</primary_host>
> >  			<primary_path>/debian</primary_path>
> >  			<primary_proto>http</primary_proto>
> > +			<url-list>
> > +				<url>
> > +					<binary>http://ftp.debian.org/debian stretch-backports main non-free contrib</binary>
> > +					<source>http://ftp.debian.org/debian stretch-backports main non-free contrib</source>
> > +				</url>
> > +				<!-- test-case for security, because Release file is not in arch folder -->
> 
> This is an example. 
> Since when are the examples supposed to be test-cases ?

It's a very basic test to check if the examples that are provided are still
working.

If we can add new functionality to our examples, it's automatically tested.
That's nice! But doesn't need a comment in the example. Removed it.

> We have 2 directrories, test and tests.
> Both of these directories should go away, because they contain
> old/unused code.

i added a patch for the v2 of this series for this.

> Then we want to have a directory with xml files which are supposed to
> test things.

we are far away from testing our error handling with invalid XMLs files.
Before that we need to improve our testing regarding proxies etc.

docs/testspec.csv describes some ideas, need to update that with what we
are actually testing, what's wip and what's future. Also need to rework
the tests a bit, that they are infrastructure independent and then publish
them.

If anyone wants to help, let me know!

> At this point, i would like to see a comment why it makes sense to add
> security, and why we added backports above... (need btrfs-tools from
> backports, becasue of feature X or because this is a cool example)

i think we don't need comments at all. updated that for v2.

> > +				<url>
> > +					<binary>http://security.debian.org/ stretch/updates main contrib non-free</binary>
> > +					<source>http://security.debian.org/ stretch/updates main contrib non-free</source>
> > +				</url>
> > +			</url-list>
> >  		</mirror>
> > +
> >  		<!-- debian testing? stable? unstable?? -->
> >  		<suite>stretch</suite>
> >  	</project>
> > @@ -103,6 +115,8 @@
> >  			<pkg>wget</pkg>
> >  			<pkg>ntpdate</pkg>
> >  			<pkg>busybox</pkg>
> > +			<!-- install a package from backports -->
> > +			<pkg pin="stretch-backports">btrfs-progs</pkg>
> >  			<!-- install the libgpio built by pbuilder  -->
> >  			<pkg>libgpio1</pkg>
> >  		</pkg-list>
> > -- 
> > 2.15.1
> > 
> > 
> > _______________________________________________
> > elbe-devel mailing list
> > elbe-devel at linutronix.de
> > https://lists.linutronix.de/mailman/listinfo/elbe-devel
> 
> -- 
> Mit freundlichen Grüßen
> Torben Hohn
> 
> Linutronix GmbH
> 
> Standort: Bremen
> 
> Phone: +49 7556 25 999 18; Fax.: +49 7556 25 999 99
> 
> Firmensitz / Registered Office: D-88690 Uhldingen, Bahnhofstr. 3
> Registergericht / Local District Court: Amtsgericht Freiburg i. Br.; HRB
> Nr. / Trade register no.: 700 806
> 
> Geschäftsführer / Managing Directors: Heinz Egger, Thomas Gleixner
> 
> Eine Bitte von uns: Sollten Sie diese E-Mail irrtümlich erhalten haben,
> benachrichtigen Sie uns in diesem Falle bitte sobald wie es Ihnen
> möglich ist, durch Antwort-Mail. Vielen Dank!





More information about the elbe-devel mailing list