After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 711184 - Patches to run in gnome-continuous
Patches to run in gnome-continuous
Status: RESOLVED WONTFIX
Product: pitivi
Classification: Other
Component: General
Git
Other Linux
: Normal normal
: Git
Assigned To: Pitivi maintainers
Pitivi maintainers
Depends on:
Blocks:
 
 
Reported: 2013-10-31 00:00 UTC by Colin Walters
Modified: 2013-10-31 01:16 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
0001-Drop-dependency-on-PyXDG-use-GLib-instead.patch (5.75 KB, patch)
2013-10-31 00:00 UTC, Colin Walters
none Details | Review
0002-Attempt-to-make-NumPy-a-soft-dependency-again.patch (2.93 KB, patch)
2013-10-31 00:01 UTC, Colin Walters
none Details | Review
0003-Drop-use-of-old-glib-module.patch (2.71 KB, patch)
2013-10-31 00:01 UTC, Colin Walters
committed Details | Review
updated xdg patch (5.74 KB, patch)
2013-10-31 00:03 UTC, Colin Walters
committed Details | Review

Description Colin Walters 2013-10-31 00:00:38 UTC
The NumPy patch is a hack, comments/suggestions appreciated.  Hopefully you guys are OK with it being a soft dependency; it's only a few lines of code.

The other two patches should be good, but let me know what you think!
Comment 1 Colin Walters 2013-10-31 00:00:54 UTC
Created attachment 258622 [details] [review]
0001-Drop-dependency-on-PyXDG-use-GLib-instead.patch
Comment 2 Colin Walters 2013-10-31 00:01:06 UTC
Created attachment 258623 [details] [review]
0002-Attempt-to-make-NumPy-a-soft-dependency-again.patch
Comment 3 Colin Walters 2013-10-31 00:01:20 UTC
Created attachment 258624 [details] [review]
0003-Drop-use-of-old-glib-module.patch
Comment 4 Colin Walters 2013-10-31 00:03:46 UTC
Created attachment 258625 [details] [review]
updated xdg patch

Ooops, a quick self-review on patch 1 caught an obvious bug in xdg_data_home().  Should be fixed now.
Comment 5 Thibault Saunier 2013-10-31 00:11:44 UTC
Review of attachment 258625 [details] [review]:

I guess you tested, in which case, this patch looks right to me.
Comment 6 Thibault Saunier 2013-10-31 00:13:39 UTC
Review of attachment 258623 [details] [review]:

Actually I had forgoten that we now depend on NumPy for the waveform generator, I think we do want it to be a hard dep rather than making it a soft now as waveform is a key feature of Pitivi. Is that really an issue for you?
Comment 7 Thibault Saunier 2013-10-31 00:14:37 UTC
Review of attachment 258624 [details] [review]:

Indeed, we should not use that at all!
Comment 8 Colin Walters 2013-10-31 01:12:34 UTC
(In reply to comment #6)
> Review of attachment 258623 [details] [review]:
> 
> Actually I had forgoten that we now depend on NumPy for the waveform generator,
> I think we do want it to be a hard dep rather than making it a soft now as
> waveform is a key feature of Pitivi. Is that really an issue for you?

The goal of the system is to produce a small, well tested core operating system using GNOME technologies (i.e. revolve around GLib/GTK+), that can then be combined into larger collections such as Debian and Red Hat Enterprise Linux.

There's nothing *wrong* with NumPy, but I want to keep the scope of the system small; there's too many duplicative libraries as is (like NSPR).

So...let's just go with the other two patches.