Re: Meld 1.8.1 released



If the exit code won't ever be usable and some clients don't have a need to wait for the process to exit, I guess I could make two wrapper executables; one that calls python and one that calls pythonw.  I did it the way I did because it was simpler (both for myself and for the caller).  Do you think that's overcomplicated?

-Keegan


On Tue, Sep 24, 2013 at 10:10 AM, Keegan Witt <keeganwitt gmail com> wrote:
That's true, but wouldn't you want tools like that to be aware of when the merging process has ended?  If I call pythonw, it will appear to the tool calling the process that the process ended immediately.  This also means the status code returned by pythonw is worthless since it will always be 0.  Isn't that an issue for these tools as well?  Sorry for my ignorance, I don't use Meld for that.

I agree that it stinks that a separate window will now be opened, but otherwise the calling process loses information about the status of the process it called.  I was thinking too of how this might be in a script that helps a user do some larger workflow.  If it exits immediately (and/or with a worthless status code), the script wouldn't know whether to proceed or not with the next steps.

-Keegan


On Tue, Sep 24, 2013 at 9:24 AM, Angel Ezquerra <angel ezquerra gmail com> wrote:
Hi,

personally I think this change (calling python.exe rather than
pythonw.exe) will be a serious regression when using meld with a tool
such as TortoiseHg. TortoiseHg _always_ calls meld with parameters.
This means that every time that you would use meld to diff or merge
files from TortoiseHg (or any similar tool) you'd see a prompt window
appear.

Cheers,

Angel



On Tue, Sep 24, 2013 at 10:35 AM, Keegan Witt <keeganwitt gmail com> wrote:
> Sorry for the delay.  Someone reported that GTK wouldn't load for him and I
> was hoping to track that down why before updating the binaries.  I haven't
> figured it out (it doesn't help that I've not been able to recreate the
> issue), but I've decided to go ahead and release new ones in the mean time.
> Has anyone else experienced this issue?  Have an suggestions?  I've already
> tried having him clear his path before calling the Python from Portable
> Python with the absolute paths (which didn't help), though he is able to run
> the GTK demo.
>
> Besides the update to 1.8.1, I also now call python.exe instead of
> pythonw.exe when calling meld.exe with parameters.  This allows you to call
> Meld from the commandline without pythonw exiting right away (the only
> downside being that a dialog box for python appears in that case -- not sure
> there's really a way around this).  When calling without parameters (for
> example from shortcuts), it continues to call pythonw.  The issue list for
> this installer release can be seen here.
>
> -Keegan
>
>
> On Sat, Sep 21, 2013 at 7:08 PM, Kai Willadsen <kai willadsen gmail com>
> wrote:
>>
>> Meld 1.8.1 has been released.
>>
>>
>>   Fixes:
>>
>>     * Add AppData file (Kai Willadsen)
>>     * Change order of version control selection for CVS and old SVN (Kai
>>       Willadsen)
>>     * Fix escaped markup in folder comparisons (Kai Willadsen)
>>
>>   Translations:
>>
>>     * Daniel Mustieles (es)
>>     * Enrico Nicoletto (pt_BR)
>>     * Gabor Kelemen (hu)
>>     * Marek Černocký (cs)
>>     * Milo Casagrande (it)
>>     * Piotr Drąg (pl)
>>
>>
>> This release can be downloaded from:
>>
>> http://download.gnome.org/sources/meld/1.8/meld-1.8.1.tar.xz
>>
>>
>> What is Meld?
>> -------------
>>
>> Meld is a visual diff and merge tool. It lets you compare two or three
>> files,
>> and updates the comparisons while you edit them in-place. You can also
>> compare
>> folders, launching comparisons of individual files as desired. Last but by
>> no
>> means least, Meld lets you work with your current changes in a wide
>> variety of
>> version control systems, including Git, Bazaar, Mercurial, Subversion and
>> CVS.
>> _______________________________________________
>> meld-list mailing list
>> meld-list gnome org
>> https://mail.gnome.org/mailman/listinfo/meld-list
>
>
>
> _______________________________________________
> meld-list mailing list
> meld-list gnome org
> https://mail.gnome.org/mailman/listinfo/meld-list




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]