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 642939 - deprecation warning from some script-fu scripts
deprecation warning from some script-fu scripts
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: Script-Fu
git master
Other Linux
: Normal normal
: 2.8
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2011-02-22 04:11 UTC by Carol Spears
Modified: 2011-11-09 18:03 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
alpha selection to item selection, drawable perspective transform to item (3.03 KB, patch)
2011-02-22 04:11 UTC, Carol Spears
none Details | Review
Fix error in camouflage plugin (3.81 KB, patch)
2011-02-22 13:06 UTC, Nelson A. de Oliveira
committed Details | Review
Updates the usage of deprecated gimp-image-add-layer (27.49 KB, patch)
2011-02-22 13:07 UTC, Nelson A. de Oliveira
committed Details | Review
deprecated removed, contexts set and a few miscelaneous repairs (70.42 KB, patch)
2011-02-23 08:53 UTC, Carol Spears
reviewed Details | Review

Description Carol Spears 2011-02-22 04:11:54 UTC
Created attachment 181553 [details] [review]
alpha selection to item selection, drawable perspective transform to item

plug-ins/script-fu/scripts/drop-shadow.scm
plug-ins/script-fu/scripts/perspective-shadow.scm
plug-ins/script-fu/scripts/xach-effect.scm

In gimp-2.7.2 these scripts function but give warnings that some procedures
they use have been deprecated.

The deprecated warnings should be removed.  There should be a patch attached
which accomplishes this.
Comment 1 Nelson A. de Oliveira 2011-02-22 13:06:13 UTC
Created attachment 181580 [details] [review]
Fix error in camouflage plugin

Fixes an error in camouflage plugin, plus the usage of some deprecated procedures
Comment 2 Nelson A. de Oliveira 2011-02-22 13:07:49 UTC
Created attachment 181581 [details] [review]
Updates the usage of deprecated gimp-image-add-layer

Replaces the usage of the deprecated gimp-image-add-layer by gimp-image-insert-layer in plugins
Applies on top of 0001-Fix-some-errors-and-usage-of-deprecated-procedures.patch
Comment 3 Carol Spears 2011-02-23 08:53:37 UTC
Created attachment 181677 [details] [review]
deprecated removed, contexts set and a few miscelaneous repairs

This patch should update most of the logo scripts including the new context
settings.  Chalk was broke, kinda (the background was merged and cleared leaving
a silly option in the dialog) and with this patch, it isn't.

Brush updates not included with this patch.
Comment 4 Kevin Cozens 2011-03-02 07:59:31 UTC
commit 403630fd5799c68cad1330289d457ea8f5f7ed47
Author: Kevin Cozens <kcozens@cvs.gnome.org>
Date:   Wed Mar 2 02:21:50 2011 -0500

    Applied modified version of first patch for bug #642939.
    Original patch supplied by Nelson A. de Oliveira. Modified to not use the
    -1 "insanity" for invalid/none layer ID's in scripts per IRC discussion.

commit b435e55d14a5d21aa533e479838441d584b828f0
Author: Kevin Cozens <kcozens@cvs.gnome.org>
Date:   Wed Mar 2 02:55:43 2011 -0500

    Applied modified version of second patch for bug #642939.
    Original patch supplied by Nelson A. de Oliveira. Modified to not use the
    -1 "insanity" for invalid/none layer ID's in scripts per IRC discussion.
    Additional scripts were changed to not pass -1 to gimp-image-insert-layer.
Comment 5 Kevin Cozens 2011-03-02 08:00:31 UTC
Review of attachment 181580 [details] [review]:

Patch was commited with minor changes.
Comment 6 Kevin Cozens 2011-03-02 08:01:09 UTC
Review of attachment 181581 [details] [review]:

Patch was commited with minor changes.
Comment 7 Kevin Cozens 2011-11-09 07:51:02 UTC
Review of attachment 181677 [details] [review]:

Used parts of the patch which protected against changes in global context.
Comment 8 Kevin Cozens 2011-11-09 07:53:39 UTC
The Script-Fu scripts have been updated to remove the use of deprecated procedures.
Comment 9 Kevin Cozens 2011-11-09 18:03:45 UTC
Relevant commits are:

commit 27bef34bd550320706b90a4fa29130c7022b2111
Author: Kevin Cozens <kcozens@svn.gnome.org>
Date:   Wed Nov 9 02:42:34 2011 -0500

    Don't change global context and protect scripts from global context settings.

commit 05d4e788300a3a796e4b1b85b5871bad91c1e6f7
Author: Alexis Wilhelm <alexiswilhelm@gmail.com>
Date:   Sat Nov 5 20:13:02 2011 +0100

    Removed use of deprecated functions in two Script-Fu scripts (bug #647834)

commit 176773dcfe07e15c38d8a7802353d4603d75c9c6
Author: Kevin Cozens <kcozens@svn.gnome.org>
Date:   Sun Nov 6 15:55:45 2011 -0500

    Second stage in removing use of deprecated functions in Script-Fu scripts
    
    Applied most of the mega patch from br0cc0li (see bug #647834) with
    additional de-tabbing and changes in whitespace.

commit 0bd1c2f09f54359d192e190467b06bb6deea8e66
Author: Kevin Cozens <kcozens@svn.gnome.org>
Date:   Thu Nov 3 22:29:49 2011 -0400

    Fix previous script-fu deprecation commit due to overzealous update script.

commit 5e015aebbaaa663d03cc6f47a58cbeaf56b08418
Author: Kevin Cozens <kcozens@svn.gnome.org>
Date:   Sun Oct 30 02:37:26 2011 -0400

    First stage in removing use of deprecated functions in Script-Fu scripts
    
    This commit replaces the names of deprecated procedures with the name of
    the new procedure where the new procedure takes the same argument list.