[edk2-devel] [PATCH 0/3] JsonLib: jansson edk2 port

[Wu, Hao A]

> -----Original Message-----
> From: Abner Chang <abner.chang at hpe.com>
> Sent: Monday, November 23, 2020 9:47 PM
> To: devel at edk2.groups.io
> Cc: Wang, Jian J <jian.j.wang at intel.com>; Wu, Hao A <hao.a.wu at intel.com>;
> Andrew Fish <afish at apple.com>; Laszlo Ersek <lersek at redhat.com>; Leif
> Lindholm <leif at nuviainc.com>; Kinney, Michael D
> <michael.d.kinney at intel.com>; Nickle Wang <nickle.wang at hpe.com>;
> Peter O'Hanley <peter.ohanley at hpe.com>
> Subject: [PATCH 0/3] JsonLib: jansson edk2 port
> 
> This set of pactehs is sent for the comments on edk2 JSON library.
> edk2 JSON library is based on jansson open source
> (https://github.com/akheron/jansson) and wrapped as an edk2 library. edk2
> JsonLib will be used by edk2 Redfish feature drivers (not contributed yet)


Hello,

Sorry for possibly missing previous discussions on the Redfish feature.
I am wondering is it more straight-forward to put the JsonLib together with the Readfish codes under RedfishPkg?

Best Regards,
Hao Wu


> and the edk2 port of libredfish library (not contributed yet) based on DMTF
> GitHub (https://github.com/DMTF/libredfish).
> 
> You will encounterred the build error with the jansson library pointed in
> submoudle (v2.13.1) and the fix is already submitted to jansson open source
> (https://github.com/akheron/jansson/pull/558).
> 
> Signed-off-by: Abner Chang <abner.chang at hpe.com>
> 
> Cc: Jian J Wang <jian.j.wang at intel.com>
> Cc: Hao A Wu <hao.a.wu at intel.com>
> Cc: Andrew Fish <afish at apple.com>
> Cc: Laszlo Ersek <lersek at redhat.com>
> Cc: Leif Lindholm <leif at nuviainc.com>
> Cc: Michael D Kinney <michael.d.kinney at intel.com>
> Cc: Nickle Wang <nickle.wang at hpe.com>
> Cc: Peter O'Hanley <peter.ohanley at hpe.com>
> 
> Abner Chang (3):
>   edk2: jansson submodule of JSON library
>   MdeModulePkg/library: EDK2 port jansson library
>   MdeModulePkg: Add EDK2 port jansson library
> 
>  .gitmodules                                   |   3 +
>  MdeModulePkg/Include/Library/JsonLib.h        | 596 +++++++++++++
>  .../Library/JsonLib/JanssonCrtLibSupport.c    | 705 +++++++++++++++
>  .../Library/JsonLib/JanssonCrtLibSupport.h    | 198 +++++
>  MdeModulePkg/Library/JsonLib/JsonLib.c        | 805 ++++++++++++++++++
>  MdeModulePkg/Library/JsonLib/JsonLib.inf      |  81 ++
>  MdeModulePkg/Library/JsonLib/JsonUtilities.c  | 417 +++++++++
> MdeModulePkg/Library/JsonLib/JsonUtilities.h  |  69 ++
>  MdeModulePkg/Library/JsonLib/assert.h         |  11 +
>  MdeModulePkg/Library/JsonLib/errno.h          |  11 +
>  MdeModulePkg/Library/JsonLib/jansson          |   1 +
>  MdeModulePkg/Library/JsonLib/jansson_config.h |  57
> ++  .../Library/JsonLib/jansson_private_config.h  |  19 +
>  MdeModulePkg/Library/JsonLib/limits.h         |  11 +
>  MdeModulePkg/Library/JsonLib/math.h           |  11 +
>  MdeModulePkg/Library/JsonLib/stdarg.h         |  11 +
>  MdeModulePkg/Library/JsonLib/stddef.h         |  11 +
>  MdeModulePkg/Library/JsonLib/stdio.h          |  11 +
>  MdeModulePkg/Library/JsonLib/stdlib.h         |  11 +
>  MdeModulePkg/Library/JsonLib/string.h         |  11 +
>  MdeModulePkg/Library/JsonLib/sys/time.h       |  11 +
>  MdeModulePkg/Library/JsonLib/sys/types.h      |  10 +
>  MdeModulePkg/Library/JsonLib/time.h           |  11 +
>  MdeModulePkg/MdeModulePkg.dec                 |   7 +-
>  MdeModulePkg/MdeModulePkg.dsc                 |   4 +-
>  ReadMe.rst                                    |   1 +
>  26 files changed, 3092 insertions(+), 2 deletions(-)  create mode 100644
> MdeModulePkg/Include/Library/JsonLib.h
>  create mode 100644
> MdeModulePkg/Library/JsonLib/JanssonCrtLibSupport.c
>  create mode 100644
> MdeModulePkg/Library/JsonLib/JanssonCrtLibSupport.h
>  create mode 100644 MdeModulePkg/Library/JsonLib/JsonLib.c
>  create mode 100644 MdeModulePkg/Library/JsonLib/JsonLib.inf
>  create mode 100644 MdeModulePkg/Library/JsonLib/JsonUtilities.c
>  create mode 100644 MdeModulePkg/Library/JsonLib/JsonUtilities.h
>  create mode 100644 MdeModulePkg/Library/JsonLib/assert.h
>  create mode 100644 MdeModulePkg/Library/JsonLib/errno.h
>  create mode 160000 MdeModulePkg/Library/JsonLib/jansson
>  create mode 100644 MdeModulePkg/Library/JsonLib/jansson_config.h
>  create mode 100644
> MdeModulePkg/Library/JsonLib/jansson_private_config.h
>  create mode 100644 MdeModulePkg/Library/JsonLib/limits.h
>  create mode 100644 MdeModulePkg/Library/JsonLib/math.h
>  create mode 100644 MdeModulePkg/Library/JsonLib/stdarg.h
>  create mode 100644 MdeModulePkg/Library/JsonLib/stddef.h
>  create mode 100644 MdeModulePkg/Library/JsonLib/stdio.h
>  create mode 100644 MdeModulePkg/Library/JsonLib/stdlib.h
>  create mode 100644 MdeModulePkg/Library/JsonLib/string.h
>  create mode 100644 MdeModulePkg/Library/JsonLib/sys/time.h
>  create mode 100644 MdeModulePkg/Library/JsonLib/sys/types.h
>  create mode 100644 MdeModulePkg/Library/JsonLib/time.h
> 
> --
> 2.17.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#67850): https://edk2.groups.io/g/devel/message/67850
Mute This Topic: https://groups.io/mt/78454019/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-