Replies: 10 comments 20 replies
-
On Sun, Sep 4, 2022 at 10:40 AM Nima Talebi ***@***.***> wrote:
Hi All,
I have not had much time to do anything useful on this package, and that's
at best caused everyone some frustration I think. I have started this
discussion on how best to move forward from here. I'm starting this
discussion of to see if we can figure something out.
Again, I'm very sorry for my absence, but I'm back now and will do my best
to help out.
Good to know, Nima.
Currently, we have finished most of the work about updating SMBIOS-3.3.0,
which is implemented by Zhongze. And we are testing it, once all things are
ready, we
will merge it in the smbion-3.3.0 branch, finally rebase to the master
branch.
BTW: I have helped to review all patches several times offline.
Originally, I would like to tell you after we have finished all the work.
But for now, you have seen a bit of change. :-)
Thanks.
Lianbo
… cc: @lian-bo <https://github.com/lian-bo>, @coiby
<https://github.com/coiby>, @vt-alt <https://github.com/vt-alt>, @dsommers
<https://github.com/dsommers>, @stefanor <https://github.com/stefanor>,
@HuO50 <https://github.com/HuO50>, @mcepl <https://github.com/mcepl>
Nima
—
Reply to this email directly, view it on GitHub
<#40>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AI3EKA5KTI3YBHI2RQIFQWTV4QDZFANCNFSM6AAAAAAQEEIOEY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
On Sun, Sep 4, 2022 at 11:25 AM Nima Talebi ***@***.***> wrote:
Ah, thank you Lianbo. When you have some time, let's have a chat; I want
to make sure I do what I can to make things easier for everyone; if there's
any changed that you think we should make to how this project is set up,
access controls, permissions, groups, and so on.
Thank you, Nima.
My initial thought was to do the SMBIOS 3.3.0 update first, and fix some
critical bugs, and then a new release may be in the Mid-November.
And the next work will include(not limit):
[1] improve the test modules
[2] investigate if it's doable to add the json output in python-dmidecode
[3] keep the old xml and convert them between xml and json if possible
[4] closely follow up the upstream dmidecode project
[5] keep an eye on the SMBIOS spec and ensure supporting new features in
new SMBIOS spec
...
We can discuss it after the smbios-3.3.0 branch is ready(update to
SMBIOS-3.3.0), and you can decide if it can be merged in the master branch.
Any thoughts?
Feel free to ping me anytime!
I'm using the irc to communicate with people upstream, but I'm not sure if
you are using it too. If yes, I would suggest creating a python-dmidecode
channel in irc tool,
we can discuss the work via irc channel in the future.
Thanks.
Lianbo
… —
Reply to this email directly, view it on GitHub
<#40 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AI3EKA6AA6VOFVYRGFNOCCTV4QJCBANCNFSM6AAAAAAQEEIOEY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
Hi, I had a period ages ago where I helped co-maintain this project; I'm the one who rewrote lots of the code to get XML support (which was needed in a project I was also involved in, rteval). I am now working full time with OpenVPN, so unfortunately my time is too scarce to follow up in this project. That said, it should be possible to extract JSON out of the main parsing. If I remember the design, the XML generation happens in two stages. The first one takes the DMI data and organizes the internal data structure (A Python dict, iirc) based on the Also, iirc, these dicts are accessible via the If you have any questions about these aspects, I can try to help out. But unfortunately, I don't have much time to spare helping maintaining this project more actively. At least not before I'm tossed into a project depending on it 🙂 Also, back in the days, I had several talks with the upstream maintainer of |
Beta Was this translation helpful? Give feedback.
-
Hi, David
On Mon, Sep 5, 2022 at 6:54 PM David Sommerseth ***@***.***> wrote:
Hi,
I had a period ages ago where I helped co-maintain this project; I'm the
one who rewrote lots of the code to get XML support (which was needed in a
project, rteval
<https://git.kernel.org/pub/scm/utils/rteval/rteval.git/about/>). I am
now working full time with OpenVPN, so unfortunately my time is too scarce
to follow up in this project.
Thank you for the contribution to python-dmidecode, David, welcome. :-)
That said, it should be possible to extract JSON out of the main parsing.
If I remember the design, the XML generation happens in two stages. The
first one takes the DMI data and organizes the internal data structure (A
Python dict, iirc) based on the pymap.xml
<https://github.com/nima/python-dmidecode/blob/master/src/pymap.xml>
definition. That internal data structure is then parsed into the resulting
XML. I would expect this second step to be easily extended outputting JSON
based on the internal Python dicts.
Good news. At least this proves it might be doable in theory.
Also, iirc, these dicts are accessible via the dmidecode.QuerySection(...)
.
If you have any questions about these aspects, I can try to help out. But
unfortunately, I don't have much time to spare helping maintaining this
project more actively. At least not before I'm tossed into a project
depending on it 🙂
Great. It definitely helps us. :-)
Also, back in the days, I had several talks with the upstream maintainer
of dmidecode which this project builds on - to see if dmidecode could be
made into a library which could be used to query and retrieve information
programmatically. But back then it was very little interest to do that.
—
… Reply to this email directly, view it on GitHub
<#40 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AI3EKA2AD5IRC7NWCFWPHG3V4XGPLANCNFSM6AAAAAAQEEIOEY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
Hi, Nima If I remember correctly, two year ago Chenlizhong said that he would like to maintain the python-dmidecode with me together, but later because he had other work and had no much time to work on this project. So I had to find a right people who is interesting in and has enough time to work on the python-dmidecode, eventually, I found HuO50(Zhongze Hu hzzjoshua@gmail.com) from my friends. I have discussed with Zhongze Hu about the python-dmidecode, and he is interesting in the python-dmidecode and has much time to work on this although he was not very familar with the python-dmidecode before this. As you know, Zhongze Hu is working on SMBIOS-3.3.0 update, I only help him to finish the current job, but the main code will be implemented by him. I would suggest to add Zhongze Hu in the maintainer list after he has finished the current SMBIOS-3.3.0 update. And I will help him to do the maintainace until he could complete all important work on upstream. But anyway, you could make the dicision. :-) Thanks. |
Beta Was this translation helpful? Give feedback.
-
BTW: |
Beta Was this translation helpful? Give feedback.
-
Hi Nima & Lianbo: |
Beta Was this translation helpful? Give feedback.
-
Hi, @nima [1] [python-dmidecode]$ git rebase smbios-3.3.0 master [2] [python-dmidecode]$ git push Or [3]submit a pull request as below: Thanks |
Beta Was this translation helpful? Give feedback.
-
Hi Lianbo,
Not disturbing me at all, of course I will do that.
It's late now, but i'll do it in the morning first thing. I'll also check
the permissions, and fix your access.
Nima
…On Sun, Oct 23, 2022 at 8:44 PM lian-bo ***@***.***> wrote:
Hi, @nima <https://github.com/nima>
Sorry to disturb you again. :-) Could you help sync the branch
smbios-3.3.0 code to the master branch? Seems that I have no permission to
rebase from the smbions-3.3.0 branch to the master branch, I did it
according to the following steps and got a failure:
[1] [python-dmidecode]$ git rebase smbios-3.3.0 master
[2] [python-dmidecode]$ git push
Username for 'https://github.com': lian-bo
Password for ***@***.***':
Total 0 (delta 0), reused 0 (delta 0), pack-reused 0
remote: error: GH006: Protected branch update failed for refs/heads/master.
remote: error: At least 1 approving review is required by reviewers with
write access.
To https://github.com/nima/python-dmidecode.git
! [remote rejected] master -> master (protected branch hook declined)
error: failed to push some refs to '
https://github.com/nima/python-dmidecode.git'
Or [3]submit a pull request as below:
#44 <#44>
Thanks
Lianbo
—
Reply to this email directly, view it on GitHub
<#40 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAPODPAPPQIQ4X6OGNYYBTWEYA2ZANCNFSM6AAAAAAQEEIOEY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
Hi Lianbo,
Okasy, this seems to be due to how master has been set up (see
screenshot). I think you have access to modify this setting too. I haven't
change it yet, because I wanted to talk to you about it first. Seems like
a good idea to leave it this way, and simply get the approval from a
collaborator, what do you think?
Nima
[image: Screen Shot 2022-10-24 at 9.20.37 AM.png]
…On Sun, Oct 23, 2022 at 8:44 PM lian-bo ***@***.***> wrote:
Hi, @nima <https://github.com/nima>
Sorry to disturb you again. :-) Could you help sync the branch
smbios-3.3.0 code to the master branch? Seems that I have no permission to
rebase from the smbions-3.3.0 branch to the master branch, I did it
according to the following steps and got a failure:
[1] [python-dmidecode]$ git rebase smbios-3.3.0 master
[2] [python-dmidecode]$ git push
Username for 'https://github.com': lian-bo
Password for ***@***.***':
Total 0 (delta 0), reused 0 (delta 0), pack-reused 0
remote: error: GH006: Protected branch update failed for refs/heads/master.
remote: error: At least 1 approving review is required by reviewers with
write access.
To https://github.com/nima/python-dmidecode.git
! [remote rejected] master -> master (protected branch hook declined)
error: failed to push some refs to '
https://github.com/nima/python-dmidecode.git'
Or [3]submit a pull request as below:
#44 <#44>
Thanks
Lianbo
—
Reply to this email directly, view it on GitHub
<#40 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAPODPAPPQIQ4X6OGNYYBTWEYA2ZANCNFSM6AAAAAAQEEIOEY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
Hi All,
I have not had much time to do anything useful on this package, and that's at best caused everyone some frustration I think. I have started this discussion on how best to move forward from here. I'm starting this discussion of to see if we can figure something out.
Again, I'm very sorry for my absence, but I'm back now and will do my best to help out.
cc: @lian-bo, @coiby, @vt-alt, @dsommers, @stefanor, @HuO50, @mcepl
Nima
5 votes ·
Beta Was this translation helpful? Give feedback.
All reactions