Re: [Libosinfo] [PATCH] freedos: add freedos 1.2 test data

2018-02-14 Thread Christophe Fergeau
On Wed, Feb 14, 2018 at 12:23:51PM +0100, Christophe Fergeau wrote:
> 
> Acked-by: Christophe Fergeau 

Actually, this was missing
diff --git a/tests/test-isodetect.c b/tests/test-isodetect.c
index 1f6e5504..56dfb440 100644
--- a/tests/test-isodetect.c
+++ b/tests/test-isodetect.c
@@ -460,6 +460,12 @@ test_sled(void)
 test_one("sled");
 }

+static void
+test_freedos(void)
+{
+test_one("freedos");
+}
+
 int
 main(int argc, char *argv[])
 {
@@ -479,6 +485,7 @@ main(int argc, char *argv[])
 g_test_add_func("/isodetect/mageia", test_mageia);
 g_test_add_func("/isodetect/sles", test_sles);
 g_test_add_func("/isodetect/sled", test_sled);
+g_test_add_func("/isodetect/freedos", test_freedos);

 /* Upfront so we don't confuse valgrind */
 osinfo_entity_get_type();

and then the tests were failing because the directory hierarchy should
be tests/isodata/freedos/freedos12.
I've squashed that in your commit and pushed.


signature.asc
Description: PGP signature
___
Libosinfo mailing list
Libosinfo@redhat.com
https://www.redhat.com/mailman/listinfo/libosinfo


Re: [Libosinfo] [PATCH] freedos: add freedos-1.2

2018-02-14 Thread Christophe Fergeau
Looks good,
Acked-by: Christophe Fergeau 

On Fri, Feb 02, 2018 at 04:26:31PM +0100, Felipe Borges wrote:
> http://www.freedos.org
> ---
>  data/os/freedos.org/freedos-1.2.xml.in | 53 
> ++
>  1 file changed, 53 insertions(+)
>  create mode 100644 data/os/freedos.org/freedos-1.2.xml.in
> 
> diff --git a/data/os/freedos.org/freedos-1.2.xml.in 
> b/data/os/freedos.org/freedos-1.2.xml.in
> new file mode 100644
> index 000..c4dee6a
> --- /dev/null
> +++ b/data/os/freedos.org/freedos-1.2.xml.in
> @@ -0,0 +1,53 @@
> +
> +
> +  http://freedos.org/freedos/1.2;>
> +freedos1.2
> +<_name>FreeDOS 1.2
> +1.2
> +<_vendor>FreeDOS
> +linux
> +freedos
> +
> +2016-12-25
> +
> +
> +  <_name>FreeDOS 1.2 "Standard" Installer
> +
> +
> +  <_name>FreeDOS 1.2 "Legacy" Installer
> +
> +
> +
> +  
> +  http://www.freedos.org/download/download/FD12CD.iso
> +  
> +FD-SETUP
> +LINUX
> +  
> +
> +
> +
> +  
> +  http://www.freedos.org/download/download/FD12LGCY.iso
> +  
> +FDI-CD
> +LINUX
> +  
> +
> +
> +
> +  
> +1
> +10
> +33554432
> +209715200
> +  
> +
> +  
> +67108864
> +419430400
> +  
> +
> +  
> +
> -- 
> 2.14.3
> 
> ___
> Libosinfo mailing list
> Libosinfo@redhat.com
> https://www.redhat.com/mailman/listinfo/libosinfo


signature.asc
Description: PGP signature
___
Libosinfo mailing list
Libosinfo@redhat.com
https://www.redhat.com/mailman/listinfo/libosinfo


Re: [Libosinfo] [PATCH] freedos: add freedos-1.2

2018-02-02 Thread Daniel P . Berrangé
On Fri, Feb 02, 2018 at 04:30:11PM +0100, Felipe Borges wrote:
> I'm not sure if projects such as FreeDOS (not-commercial) are in the
> scope of libosinfo. If not, please ignore the patches.

If it is an operating system and it is available to a non-trivial
number of people, it is in scope. 

Regards,
Daniel
-- 
|: https://berrange.com  -o-https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o-https://fstop138.berrange.com :|
|: https://entangle-photo.org-o-https://www.instagram.com/dberrange :|

___
Libosinfo mailing list
Libosinfo@redhat.com
https://www.redhat.com/mailman/listinfo/libosinfo


Re: [Libosinfo] [PATCH] freedos: add freedos-1.2

2018-02-02 Thread Felipe Borges
I'm not sure if projects such as FreeDOS (not-commercial) are in the
scope of libosinfo. If not, please ignore the patches.

On Fri, Feb 2, 2018 at 4:26 PM, Felipe Borges  wrote:
> http://www.freedos.org
> ---
>  data/os/freedos.org/freedos-1.2.xml.in | 53 
> ++
>  1 file changed, 53 insertions(+)
>  create mode 100644 data/os/freedos.org/freedos-1.2.xml.in
>
> diff --git a/data/os/freedos.org/freedos-1.2.xml.in 
> b/data/os/freedos.org/freedos-1.2.xml.in
> new file mode 100644
> index 000..c4dee6a
> --- /dev/null
> +++ b/data/os/freedos.org/freedos-1.2.xml.in
> @@ -0,0 +1,53 @@
> +
> +
> +  http://freedos.org/freedos/1.2;>
> +freedos1.2
> +<_name>FreeDOS 1.2
> +1.2
> +<_vendor>FreeDOS
> +linux
> +freedos
> +
> +2016-12-25
> +
> +
> +  <_name>FreeDOS 1.2 "Standard" Installer
> +
> +
> +  <_name>FreeDOS 1.2 "Legacy" Installer
> +
> +
> +
> +  
> +  http://www.freedos.org/download/download/FD12CD.iso
> +  
> +FD-SETUP
> +LINUX
> +  
> +
> +
> +
> +  
> +  http://www.freedos.org/download/download/FD12LGCY.iso
> +  
> +FDI-CD
> +LINUX
> +  
> +
> +
> +
> +  
> +1
> +10
> +33554432
> +209715200
> +  
> +
> +  
> +67108864
> +419430400
> +  
> +
> +  
> +
> --
> 2.14.3
>

___
Libosinfo mailing list
Libosinfo@redhat.com
https://www.redhat.com/mailman/listinfo/libosinfo


[Libosinfo] [PATCH] freedos: add freedos 1.2 test data

2018-02-02 Thread Felipe Borges
---
 tests/isodata/freedos/FD12CD.iso.txt   | 29 +
 tests/isodata/freedos/FD12LGCY.iso.txt | 29 +
 2 files changed, 58 insertions(+)
 create mode 100644 tests/isodata/freedos/FD12CD.iso.txt
 create mode 100644 tests/isodata/freedos/FD12LGCY.iso.txt

diff --git a/tests/isodata/freedos/FD12CD.iso.txt 
b/tests/isodata/freedos/FD12CD.iso.txt
new file mode 100644
index 000..5205591
--- /dev/null
+++ b/tests/isodata/freedos/FD12CD.iso.txt
@@ -0,0 +1,29 @@
+CD-ROM is in ISO 9660 format
+System id: LINUX
+Volume id: FD-SETUP
+Volume set id: 
+Publisher id: 
+Data preparer id: 
+Application id: GENISOIMAGE ISO 9660/HFS FILESYSTEM CREATOR (C) 1993 
E.YOUNGDALE (C) 1997-2006 J.PEARSON/J.SCHILLING (C) 2006-2007 CDRKIT TEAM
+Copyright File id: 
+Abstract File id: 
+Bibliographic File id: 
+Volume set size is: 1
+Volume set sequence number is: 1
+Logical block size is: 2048
+Volume size is: 214247
+El Torito VD version 1 found, boot catalog is in sector 69
+NO Joliet present
+Rock Ridge signatures version 1 found
+Eltorito validation header:
+Hid 1
+Arch 0 (x86)
+ID ''
+Key 55 AA
+Eltorito defaultboot header:
+Bootid 88 (bootable)
+Boot media 0 (No Emulation Boot)
+Load segment 0
+Sys type 0
+Nsect 4
+Bootoff 46 70
diff --git a/tests/isodata/freedos/FD12LGCY.iso.txt 
b/tests/isodata/freedos/FD12LGCY.iso.txt
new file mode 100644
index 000..44ea185
--- /dev/null
+++ b/tests/isodata/freedos/FD12LGCY.iso.txt
@@ -0,0 +1,29 @@
+CD-ROM is in ISO 9660 format
+System id: LINUX
+Volume id: FDI-CD
+Volume set id: 
+Publisher id: 
+Data preparer id: 
+Application id: GENISOIMAGE ISO 9660/HFS FILESYSTEM CREATOR (C) 1993 
E.YOUNGDALE (C) 1997-2006 J.PEARSON/J.SCHILLING (C) 2006-2007 CDRKIT TEAM
+Copyright File id: 
+Abstract File id: 
+Bibliographic File id: 
+Volume set size is: 1
+Volume set sequence number is: 1
+Logical block size is: 2048
+Volume size is: 214188
+El Torito VD version 1 found, boot catalog is in sector 68
+NO Joliet present
+Rock Ridge signatures version 1 found
+Eltorito validation header:
+Hid 1
+Arch 0 (x86)
+ID ''
+Key 55 AA
+Eltorito defaultboot header:
+Bootid 88 (bootable)
+Boot media 2 (1.44MB Floppy)
+Load segment 0
+Sys type 0
+Nsect 1
+Bootoff 45 69
-- 
2.14.3

___
Libosinfo mailing list
Libosinfo@redhat.com
https://www.redhat.com/mailman/listinfo/libosinfo


[Libosinfo] [PATCH] freedos: add freedos-1.2

2018-02-02 Thread Felipe Borges
http://www.freedos.org
---
 data/os/freedos.org/freedos-1.2.xml.in | 53 ++
 1 file changed, 53 insertions(+)
 create mode 100644 data/os/freedos.org/freedos-1.2.xml.in

diff --git a/data/os/freedos.org/freedos-1.2.xml.in 
b/data/os/freedos.org/freedos-1.2.xml.in
new file mode 100644
index 000..c4dee6a
--- /dev/null
+++ b/data/os/freedos.org/freedos-1.2.xml.in
@@ -0,0 +1,53 @@
+
+
+  http://freedos.org/freedos/1.2;>
+freedos1.2
+<_name>FreeDOS 1.2
+1.2
+<_vendor>FreeDOS
+linux
+freedos
+
+2016-12-25
+
+
+  <_name>FreeDOS 1.2 "Standard" Installer
+
+
+  <_name>FreeDOS 1.2 "Legacy" Installer
+
+
+
+  
+  http://www.freedos.org/download/download/FD12CD.iso
+  
+FD-SETUP
+LINUX
+  
+
+
+
+  
+  http://www.freedos.org/download/download/FD12LGCY.iso
+  
+FDI-CD
+LINUX
+  
+
+
+
+  
+1
+10
+33554432
+209715200
+  
+
+  
+67108864
+419430400
+  
+
+  
+
-- 
2.14.3

___
Libosinfo mailing list
Libosinfo@redhat.com
https://www.redhat.com/mailman/listinfo/libosinfo