Bug 503151
| Summary: | NDesk-DBus bug using sugar datastore | ||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Torello Querci <torello> | ||||||||||||||
| Component: | ndesk-dbus | Assignee: | Peter Gordon <peter> | ||||||||||||||
| Status: | CLOSED WONTFIX | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||||||||||||
| Severity: | high | Docs Contact: | |||||||||||||||
| Priority: | low | ||||||||||||||||
| Version: | 12 | CC: | bernie+fedora, gnomeuser | ||||||||||||||
| Target Milestone: | --- | Keywords: | Patch | ||||||||||||||
| Target Release: | --- | ||||||||||||||||
| Hardware: | All | ||||||||||||||||
| OS: | Linux | ||||||||||||||||
| Whiteboard: | |||||||||||||||||
| Fixed In Version: | Doc Type: | Bug Fix | |||||||||||||||
| Doc Text: | Story Points: | --- | |||||||||||||||
| Clone Of: | Environment: | ||||||||||||||||
| Last Closed: | 2010-12-05 06:53:17 UTC | Type: | --- | ||||||||||||||
| Regression: | --- | Mount Type: | --- | ||||||||||||||
| Documentation: | --- | CRM: | |||||||||||||||
| Verified Versions: | Category: | --- | |||||||||||||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||||||||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||||||||||||
| Embargoed: | |||||||||||||||||
| Attachments: |
|
||||||||||||||||
|
Description
Torello Querci
2009-05-29 09:32:36 UTC
Created attachment 345901 [details]
ndesk-dbus.patch
for reference since you forgot, these are the patches you sent to me in private. Once I get my Fedora account fixed up I will commit these to Rawhide and if nothing breaks wildly I am more than open to do the requested push to the stable platforms.
Created attachment 345902 [details]
ndesk-dbus-glib.patch
David, what's wrong with your FAS account? If you can't get it sorted out soon, I can offer to commit these changes for you (I'm a Fedora provenpackager). cvs keeps claiming I don't have rights even to checkout things. Regardless I would be happy if you would take the time to do this work, I imagine the sugar lab guys would be happy if that happened and I can't see myself having the time to mess with FAS for a while. (In reply to comment #4) > cvs keeps claiming I don't have rights even to checkout things. Regardless I > would be happy if you would take the time to do this work, I imagine the sugar > lab guys would be happy if that happened and I can't see myself having the time > to mess with FAS for a while. I'm looking into it. Meanwhile, you could go to #fedora-admin on irc.freenode.net and ask for help with your account. This bug appears to have been reported against 'rawhide' during the Fedora 11 development cycle. Changing version to '11'. More information and reason for this action is here: http://fedoraproject.org/wiki/BugZappers/HouseKeeping I recently took over ownership of these two NDesk packages in Fedora; and I definitely reproduce the bug as Torello mentions in the first post to this report. However, the ndesk-dbus patch as-is don't apply properly, and I'm having trouble figuring out how to rediff them properly (for example, it looks like most of the ReadMessage code has been moved into src/Connection.cs, and the ReadMessageReal hack is no longer present at all. The ndesk-dbus-glib patch does apply correctly; but since it requires the WaitForIOCompletion stuff from the ndesk-dbus one, can't build yet. Would you please attached an updated ndesk-dbus patch? I'd appreciate that very much. Thanks. Created attachment 355566 [details]
Right patch for ndesk-dbus-0.6.1a
This patch related with 0.6.1a version of thi library.
Created attachment 355569 [details]
Spec file with patch for ndesk-dbus
Created attachment 355570 [details]
Spec file with patch for ndesk-dbus-glib
Sorry Peter, I see this message only today. The right patch file was present on http://code.google.com/p/sugar-sharp/ website. I already prepared the specfile with the patch. I just upload the right (I hope) file here. If I can do anything simple ask me :) Best regards Thanks for the updated patch! I've committed and built these patches into Rawhide. After a few days there, to help ensure it doesn't cause any regressions or negative issues, I'll push updates for them to Fedora 10/11. Regards. Note that there are spelling errors in the patch. Also, has upstream been contacted at all? While there have been no release since 2008, the Git repository is still being periodically updated. I have filed a bug upstream, recommending that the changes made by both mono-zeroconf and sugar-sharp be merged (and the API version bumped). This bug appears to have been reported against 'rawhide' during the Fedora 12 development cycle. Changing version to '12'. More information and reason for this action is here: http://fedoraproject.org/wiki/BugZappers/HouseKeeping I sent the bug report and the patch at April 2009. The mantainer told me that the problem is present and work to fix it, but today I check the git code and the bug it is still present. I not know why this patch is not adopted. Next I check my with mono-zeroconf to verify if the library/patch can be joined. Sorry, I tested the git version again and with the following changes it now works.
I sent an email to the maintainer
Only one problem remains and this patch fixes it:
--- a/src/MessageReader.cs
+++ b/src/MessageReader.cs
@@ -420,7 +420,7 @@ namespace NDesk.DBus
}
//advance to the alignment of the element
- ReadPad (Protocol.GetAlignment (Signature.TypeToDType (elemType)));
+ ReadPad (Protocol.GetAlignment (DType.Array));
int endPos = pos + (int)ln;
This message is a reminder that Fedora 12 is nearing its end of life. Approximately 30 (thirty) days from now Fedora will stop maintaining and issuing updates for Fedora 12. It is Fedora's policy to close all bug reports from releases that are no longer maintained. At that time this bug will be closed as WONTFIX if it remains open with a Fedora 'version' of '12'. Package Maintainer: If you wish for this bug to remain open because you plan to fix it in a currently maintained version, simply change the 'version' to a later Fedora version prior to Fedora 12's end of life. Bug Reporter: Thank you for reporting this issue and we are sorry that we may not be able to fix it before Fedora 12 is end of life. If you would still like to see this bug fixed and are able to reproduce it against a later version of Fedora please change the 'version' of this bug to the applicable version. If you are unable to change the version, please add a comment here and someone will do it for you. Although we aim to fix as many bugs as possible during every release's lifetime, sometimes those efforts are overtaken by events. Often a more recent Fedora release includes newer upstream software that fixes bugs or makes them obsolete. The process we are following is described here: http://fedoraproject.org/wiki/BugZappers/HouseKeeping Fedora 12 changed to end-of-life (EOL) status on 2010-12-02. Fedora 12 is no longer maintained, which means that it will not receive any further security or bug fix updates. As a result we are closing this bug. If you can reproduce this bug against a currently maintained version of Fedora please feel free to reopen this bug against that version. Thank you for reporting this bug and we are sorry it could not be fixed. |