GGR from Image

[Ofnuts]

(10-07-2022, 08:50 AM)rich2005 Wrote: To make it more portable comment out line 39 and add a line as;

Code:

gradient_folder = os.path.join(gimp.directory, "gradients", "")

That is working here, linux Gimp 2.10.32 also in a Win10 VM both for regular Gimp and Samj portable.

Definitely the way... works for any OS, Gimp version, and GIMP2_DIRECTORYenvironment setting.

For the nitpicking crowd, the final ,'' is not necessary, it is used to add a final /, but this thing should be done else where, for instance

Code:

newfilename = gradient_folder + gradient_name + ".svg"

should be

Code:

newfilename = os.path.join(gradient_folder,gradient_name + ".svg")

(lines 67, 107 and 175) and the various calls to os.path.expanduser() are not necessary because the directory returned by gimp.directory() is already fully resolved. In fact one wonders why this was not done once for all when initializing the gradient_folder variable, and the DRY principle demands that you would have some gradientFileFromName() function that would handle all this in a single place.