* bug#32359: [PATCH] Add svg-path @ 2018-08-03 11:07 Felix E. Klee 2018-08-03 13:05 ` Eli Zaretskii 0 siblings, 1 reply; 27+ messages in thread From: Felix E. Klee @ 2018-08-03 11:07 UTC (permalink / raw) To: 32359 [-- Attachment #1: Type: text/plain, Size: 215 bytes --] The patch adds `svg-path'. Among other things, this function makes it possible to add arcs to SVG images. A path is drawn using commands defined by the SVG standard: https://www.w3.org/TR/SVG11/paths.html#PathData [-- Attachment #2: add-svg-path.patch --] [-- Type: application/octet-stream, Size: 1784 bytes --] diff --git a/ChangeLog.3 b/ChangeLog.3 index a0a4794b4e..2a9832b67b 100644 --- a/ChangeLog.3 +++ b/ChangeLog.3 @@ -1,3 +1,7 @@ +2018-08-03 Felix E. Klee <felix.klee@inka.de> + + * lisp/svg.el (svg-path): New function. + 2018-07-01 Paul Eggert <eggert@cs.ucla.edu> * etc/HISTORY: Cite Brinkhoff on early history. diff --git a/doc/lispref/display.texi b/doc/lispref/display.texi index fef5188197..d45f68d95b 100644 --- a/doc/lispref/display.texi +++ b/doc/lispref/display.texi @@ -5541,6 +5541,19 @@ that describe the outer circumference of the polygon. @end lisp @end defun +@defun svg-path svg commands &rest args +Add the outline of a shape to @var{svg}. The @var{commands} follow the +Scalable Vector Graphics standard. This function can be used to create +arcs. + +@lisp +(svg-path svg '((M 100 300) + (A 300 300 0 0 0 300 100) + (Z)) + :stroke-color "blue" :fill-color "yellow") +@end lisp +@end defun + @defun svg-text svg text &rest args Add a text to @var{svg}. diff --git a/lisp/svg.el b/lisp/svg.el index 1178905546..e5c4e1383a 100644 --- a/lisp/svg.el +++ b/lisp/svg.el @@ -138,6 +138,17 @@ POINTS is a list of x/y pairs." ", ")) ,@(svg--arguments svg args))))) +(defun svg-path (svg commands &rest args) + "Add the outline of a shape to SVG. The COMMANDS follow the +Scalable Vector Graphics standard. This function can be used to +create arcs." + (let ((d (mapconcat 'prin1-to-string (apply 'append commands) " "))) + (svg--append + svg + (dom-node 'path + `((d . ,d) + ,@(svg--arguments svg args)))))) + (defun svg-embed (svg image image-type datap &rest args) "Insert IMAGE into the SVG structure. IMAGE should be a file name if DATAP is nil, and a binary string ^ permalink raw reply related [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2018-08-03 11:07 bug#32359: [PATCH] Add svg-path Felix E. Klee @ 2018-08-03 13:05 ` Eli Zaretskii 2018-08-23 15:50 ` Felix E. Klee 0 siblings, 1 reply; 27+ messages in thread From: Eli Zaretskii @ 2018-08-03 13:05 UTC (permalink / raw) To: Felix E. Klee; +Cc: 32359 > From: "Felix E. Klee" <felix.klee@inka.de> > Date: Fri, 3 Aug 2018 13:07:42 +0200 > > The patch adds `svg-path'. Among other things, this function makes it > possible to add arcs to SVG images. A path is drawn using commands > defined by the SVG standard: > > https://www.w3.org/TR/SVG11/paths.html#PathData Thanks. This contribution can be accepted without legal paperwork, but the next one will need a copyright assignment, so I'd encourage you to start your paperwork now. A few comments, mainly about the documentation parts. > diff --git a/ChangeLog.3 b/ChangeLog.3 > index a0a4794b4e..2a9832b67b 100644 > --- a/ChangeLog.3 > +++ b/ChangeLog.3 > @@ -1,3 +1,7 @@ > +2018-08-03 Felix E. Klee <felix.klee@inka.de> > + > + * lisp/svg.el (svg-path): New function. > + We don't maintain a ChangeLog file; the above should be the commit log message. > +@defun svg-path svg commands &rest args > +Add the outline of a shape to @var{svg}. The @var{commands} follow the > +Scalable Vector Graphics standard. This function can be used to create > +arcs. This is too cryptic for the manual, and the example doesn't help enough. We should at least explain what "arcs" means in this context, and in general what is this function about; also what kind of object is SCG. Also, please observe our standard of having 2 spaces between sentences. In general, GNU Coding Standards frown upon using "path" for anything that is not PATH-style directory lists, so maybe use a different name or explain what kind of "path" is being referenced here. E.g., the Web page to which you pointed does include a definition of "path" in this context. > +(defun svg-path (svg commands &rest args) > + "Add the outline of a shape to SVG. The COMMANDS follow the > +Scalable Vector Graphics standard. This function can be used to > +create arcs." The first line of the doc string should be a single complete sentence, and it should mention all of the arguments. Also, please keep 2 spaces between sentences in the doc strings. I also think the doc string should say that COMMANDS are strings, or objects whose printed representation yields valid SVG commands. This new function should also be announced in NEWS. ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2018-08-03 13:05 ` Eli Zaretskii @ 2018-08-23 15:50 ` Felix E. Klee 2018-08-23 17:23 ` Eli Zaretskii 2019-06-23 22:15 ` Lars Ingebrigtsen 0 siblings, 2 replies; 27+ messages in thread From: Felix E. Klee @ 2018-08-23 15:50 UTC (permalink / raw) To: Eli Zaretskii; +Cc: 32359 Hi Eli, thanks for the feedback! On 8/3/18, Eli Zaretskii <eliz@gnu.org> wrote: > We don't maintain a ChangeLog file; the above should be the commit > log message. Well, the Emacs info page on committing patches states: “Write the change log entries for your changes. […]” Furthermore, it links to a page explaining `ChangeLog' files in particular. Is that documentation outdated? `ChangeLog.3' contains entries for previous additions to `svg.el', the latest one as recent as September 2017. Anyhow, if you want a commit log instead of a `ChangeLog' entry, how should I submit the commit? >> +@defun svg-path svg commands &rest args >> +Add the outline of a shape to @var{svg}. The @var{commands} follow the >> +Scalable Vector Graphics standard. This function can be used to create >> +arcs. > > This is too cryptic for the manual, It’s basically in line with the description for the other `svg' functions. > and the example doesn't help enough. For the intended audience, i.e. those knowing how to author SVG documents, it should be clear. What I could do is add aliases for the path commands: * `moveto-relative' → `m' * `moveto-absolute' → `M' * etc. The example would then turn into: (svg-path svg '((moveto-absolute 100 300) (arc-absolute 300 300 0 0 0 300 100) (closepath-absolute)) :stroke-color "blue" :fill-color "yellow") Still users would need to know the command parameters which are detailed in every comprehensive documentation about SVG. > We should at least explain what "arcs" means in this context, “arcs” in the context of SVG, i.e. vector graphics refers to curved paths: https://www.merriam-webster.com/dictionary/arc > and in general what is this function about; also what kind of object > is SCG. Quote from the documentation in the elisp info page for `svg.el': “SVG (Scalable Vector Graphics) is an XML format for specifying images.” > In general, GNU Coding Standards frown upon using "path" for > anything that is not PATH-style directory lists, so maybe use a > different name or explain what kind of "path" is being referenced > here. Renaming `path' would be super confusing to those familiar with vector graphics. If `path' needs to be avoided, then I’d rather not add `svg-path'. Instead one could add a more general: svg-node (parent tag &rest args) This function is needed anyhow. It would be for inserting custom SVG nodes, i.e. nodes beyond those currently available in the `svg' package. BTW I got in contact with Lars, the original author of `svg.el', and he’s OK with the changes I proposed, including some additional functions. / Felix ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2018-08-23 15:50 ` Felix E. Klee @ 2018-08-23 17:23 ` Eli Zaretskii 2018-08-24 9:15 ` Felix E. Klee 2019-06-23 22:15 ` Lars Ingebrigtsen 1 sibling, 1 reply; 27+ messages in thread From: Eli Zaretskii @ 2018-08-23 17:23 UTC (permalink / raw) To: Felix E. Klee; +Cc: 32359 > From: "Felix E. Klee" <felix.klee@inka.de> > Date: Thu, 23 Aug 2018 17:50:08 +0200 > Cc: 32359@debbugs.gnu.org > > thanks for the feedback! Thanks for working on improving Emacs. > > We don't maintain a ChangeLog file; the above should be the commit > > log message. > > Well, the Emacs info page on committing patches states: “Write the > change log entries for your changes. […]” The relevant place to look nowadays is CONTRIBUTE in the top-level directory of the Emacs tree. > Furthermore, it links to a page explaining `ChangeLog' files in > particular. Is that documentation outdated? It says "change logs", not ChangeLog. However, since it confused you, I have now replaced that with "commit log". The reference to the GNU Coding Standards is still relevant, I think, because we still use the ChangeLog style in our commit log messages. > `ChangeLog.3' contains entries for previous additions to `svg.el', the > latest one as recent as September 2017. That file is auto-generated nowadays from Git log. > Anyhow, if you want a commit log instead of a `ChangeLog' entry, how > should I submit the commit? The best is in "git format-patch" format, which will include your commit log message. If that's hard or complicated for you, a normal patch will do with the commit log as preamble, i.e. not in Diff format. > >> +@defun svg-path svg commands &rest args > >> +Add the outline of a shape to @var{svg}. The @var{commands} follow the > >> +Scalable Vector Graphics standard. This function can be used to create > >> +arcs. > > > > This is too cryptic for the manual, > > It’s basically in line with the description for the other `svg' functions. Maybe so, but there's no reason to continue that practice when we add new functions. It is also okay to fix documentation of those other functions, but of course it doesn't have to be part of this particular changeset. > > and the example doesn't help enough. > > For the intended audience, i.e. those knowing how to author SVG > documents, it should be clear. Okay, but the manual exists not only for those already "in the know". It should also cater to those who are studying the subject with the purpose of writing some code for the first time in this domain. > What I could do is add aliases for the path commands: > > * `moveto-relative' → `m' > > * `moveto-absolute' → `M' > > * etc. I think it would be better simply to explain those in plain English, not by adding code. > The example would then turn into: > > (svg-path svg '((moveto-absolute 100 300) > (arc-absolute 300 300 0 0 0 300 100) > (closepath-absolute)) > :stroke-color "blue" :fill-color "yellow") > > Still users would need to know the command parameters which are > detailed in every comprehensive documentation about SVG. Exactly. So it's slightly better (less cryptic), but still not clear enough, because the meaning of moveto-relative and moveto-absolute is not entirely obvious, only their general idea is evident ("relative" vs "absolute"). I understand that it isn't reasonable to have the entire SVG docs there, but we should explain a bit more before pointing to the official SVG docs for further details. I trust you that you will know where to draw the line. > > We should at least explain what "arcs" means in this context, > > “arcs” in the context of SVG, i.e. vector graphics refers to curved > paths: > > https://www.merriam-webster.com/dictionary/arc I didn't mean explain to me, I meant explain in the manual. > > and in general what is this function about; also what kind of object > > is SCG. > > Quote from the documentation in the elisp info page for `svg.el': “SVG > (Scalable Vector Graphics) is an XML format for specifying images.” We are mis-communicating. I didn't mean SVG the acronym, I meant SVG the argument of the function. The doc strings says Add the outline of a shape to SVG. It is quite clear that "SVG" here doesn't stand for Scalable Vector Graphics, it stands for some object to which the function will add a shape. I'm asking to say a word or two about what kind of object is that, from the Lisp program POV. is it a string? a symbol? a list? a buffer? something else? > > In general, GNU Coding Standards frown upon using "path" for > > anything that is not PATH-style directory lists, so maybe use a > > different name or explain what kind of "path" is being referenced > > here. > > Renaming `path' would be super confusing to those familiar with vector > graphics. If `path' needs to be avoided, then I’d rather not add > `svg-path'. There's the second alternative: explain in a few words what kind of "path" is meant here. Then you can use the word freely. > BTW I got in contact with Lars, the original author of `svg.el', and > he’s OK with the changes I proposed, including some additional > functions. That's fine, but the way our patch review process works, the comments are additive ;-) Thanks. ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2018-08-23 17:23 ` Eli Zaretskii @ 2018-08-24 9:15 ` Felix E. Klee 0 siblings, 0 replies; 27+ messages in thread From: Felix E. Klee @ 2018-08-24 9:15 UTC (permalink / raw) To: Eli Zaretskii; +Cc: 32359 On Thu, Aug 23, 2018 at 7:23 PM, Eli Zaretskii <eliz@gnu.org> wrote: > I think it would be better simply to explain those in plain English, > not by adding code. OK. Still the aliases do increase readability. They are not made up. They are based on the “names” listed in the [specs][1]. For now, I am considering creating a separate package `svg+.el'. This would allow me to play with various approaches, possibly with user feedback. If that all works, one could merge the additions into Emacs. [1]: https://www.w3.org/TR/SVG11/paths.html ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2018-08-23 15:50 ` Felix E. Klee 2018-08-23 17:23 ` Eli Zaretskii @ 2019-06-23 22:15 ` Lars Ingebrigtsen 2019-06-24 14:32 ` Felix E. Klee 1 sibling, 1 reply; 27+ messages in thread From: Lars Ingebrigtsen @ 2019-06-23 22:15 UTC (permalink / raw) To: Felix E. Klee; +Cc: 32359 "Felix E. Klee" <felix.klee@inka.de> writes: > The example would then turn into: > > (svg-path svg '((moveto-absolute 100 300) > (arc-absolute 300 300 0 0 0 300 100) > (closepath-absolute)) > :stroke-color "blue" :fill-color "yellow") > > Still users would need to know the command parameters which are > detailed in every comprehensive documentation about SVG. I think this looks very nice -- the svg library is quite close to the SVG specification, but when it makes sense to clarify some of the more cryptic bits with clearer names, it should. So these are better names for an svg path than M/A/Z, which are extremely obscure. So I'd welcome the patch with these name mappings. -- (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-06-23 22:15 ` Lars Ingebrigtsen @ 2019-06-24 14:32 ` Felix E. Klee 2019-06-24 14:39 ` Lars Ingebrigtsen 0 siblings, 1 reply; 27+ messages in thread From: Felix E. Klee @ 2019-06-24 14:32 UTC (permalink / raw) To: Lars Ingebrigtsen; +Cc: 32359 Please have a look at: https://github.com/feklee/svg-plus Now the example is: (svg+-path svg '((moveto ((100 . 300))) (elliptical-arc ((300 300 0 0 :large-arc nil :sweep nil))) (closepath :relative nil)) :stroke-color "blue" :fill-color "yellow") I wanted to announce this package once it’s finished, which is always a bad idea. IIRC only complete documentation is missing, and of course more SVG features could be added. On Wednesday there is the Emacs meetup in Berlin, and indeed I was thinking about presenting the package there. So that could be some motivation to finish it, and then we can talk about a patch. Thanks for bringing it up again. :) ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-06-24 14:32 ` Felix E. Klee @ 2019-06-24 14:39 ` Lars Ingebrigtsen 2019-07-12 14:35 ` Felix E. Klee 0 siblings, 1 reply; 27+ messages in thread From: Lars Ingebrigtsen @ 2019-06-24 14:39 UTC (permalink / raw) To: Felix E. Klee; +Cc: 32359 "Felix E. Klee" <felix.klee@inka.de> writes: > Please have a look at: > > https://github.com/feklee/svg-plus > > Now the example is: > > (svg+-path svg > '((moveto ((100 . 300))) > (elliptical-arc ((300 300 0 0 :large-arc nil :sweep nil))) > (closepath :relative nil)) > :stroke-color "blue" > :fill-color "yellow") That's really good -- makes it actually understandable how a path is composed. > I wanted to announce this package once it’s finished, which is always a > bad idea. IIRC only complete documentation is missing, and of course > more SVG features could be added. On Wednesday there is the Emacs > meetup in Berlin, and indeed I was thinking about presenting the package > there. So that could be some motivation to finish it, and then we can > talk about a patch. Bringing that function into svg.el in Emacs would be nice. -- (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-06-24 14:39 ` Lars Ingebrigtsen @ 2019-07-12 14:35 ` Felix E. Klee 2019-07-12 16:02 ` Lars Ingebrigtsen 0 siblings, 1 reply; 27+ messages in thread From: Felix E. Klee @ 2019-07-12 14:35 UTC (permalink / raw) To: Lars Ingebrigtsen; +Cc: 32359 Hi Lars, finally I got around to finishing documentation. Find the latest release at: https://github.com/feklee/svg-plus In addition to `svg+-path`, I added: * `svg+-clip-path` * `svg+-node` If you want to play with the new functions, simply run the code examples from the info page inside of `example.el`. Concerning `svg+-path`, I’d like to hear your thoughts: Do you see room for further improvement, or could this be the final interface? I was thinking about removing the double parentheses in commands such as: (lineto ((100 . 200))) One could simplify that to: (lineto (100 . 200)) Polylines would change from: (lineto ((100 . 200) (300 . 200))) To: (lineto (100 . 200) (300 . 200)) Reasons against this change: * Parsing optional arguments (plist) could be a pain. Currently what works nicely: (lineto ((100 . 200) (200 . 0)) :relative t) Elliptical arc allows additional optional arguments, such as: (elliptical-arc ((50 100 200 300 :large-arc 1)) :relative t) * Aesthetically, some standard Lisp commands aren’t “better.” For example `let` also requires double parentheses for single assignments: (let ((x 3)) (* x x)) ―Felix ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-07-12 14:35 ` Felix E. Klee @ 2019-07-12 16:02 ` Lars Ingebrigtsen 2019-07-15 13:34 ` Felix E. Klee 0 siblings, 1 reply; 27+ messages in thread From: Lars Ingebrigtsen @ 2019-07-12 16:02 UTC (permalink / raw) To: Felix E. Klee; +Cc: 32359 "Felix E. Klee" <felix.klee@inka.de> writes: > If you want to play with the new functions, simply run the code examples > from the info page inside of `example.el`. Concerning `svg+-path`, I’d > like to hear your thoughts: Do you see room for further improvement, or > could this be the final interface? I haven't read it closely, but it looks good to me. Could you post it as a patch to svg.el? That'll make it easier to read. > I was thinking about removing the double parentheses in commands such > as: > > (lineto ((100 . 200))) > > One could simplify that to: > > (lineto (100 . 200)) > > Polylines would change from: > > (lineto ((100 . 200) (300 . 200))) > > To: > > (lineto (100 . 200) (300 . 200)) > > Reasons against this change: > > * Parsing optional arguments (plist) could be a pain. Currently what > works nicely: > > (lineto ((100 . 200) (200 . 0)) :relative t) Yeah, I think you should keep the argument as a single argument. Varargs with optional keywords parameters just isn't very nice. -- (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-07-12 16:02 ` Lars Ingebrigtsen @ 2019-07-15 13:34 ` Felix E. Klee 2019-07-15 15:12 ` Lars Ingebrigtsen 0 siblings, 1 reply; 27+ messages in thread From: Felix E. Klee @ 2019-07-15 13:34 UTC (permalink / raw) To: Lars Ingebrigtsen; +Cc: 32359 [-- Attachment #1: Type: text/plain, Size: 239 bytes --] Hi Lars, find attached a patch against (coincidentally) your latest Emacs commit: dea9970bc0deaf320e78c46a2e7456cbb6e7a0ea Included are changes to the code and to the manual. If you need anything else, let me know! ―Felix [-- Attachment #2: 0001-Add-functions-svg-path-svg-clip-path-svg-node.patch.gz --] [-- Type: application/gzip, Size: 3877 bytes --] ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-07-15 13:34 ` Felix E. Klee @ 2019-07-15 15:12 ` Lars Ingebrigtsen 2019-07-15 20:40 ` Felix E. Klee 0 siblings, 1 reply; 27+ messages in thread From: Lars Ingebrigtsen @ 2019-07-15 15:12 UTC (permalink / raw) To: Felix E. Klee; +Cc: 32359 "Felix E. Klee" <felix.klee@inka.de> writes: > Included are changes to the code and to the manual. If you need > anything else, let me know! Thanks! It looks great, so I applied it to the trunk. The only possible tweak was with using moveto vs. move-to. The latter looks more Emacsish, but in SVG speak they're moveto (etc), so I think it makes sense to leave them that way. But after applying, I was suddenly unsure whether you'd signed FSF copyright assignment papers yet or not. I thought I remembered seeing your name in the copyright.list file before, but now I can't see it. Have you gone through the paperwork process with the FSF? -- (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-07-15 15:12 ` Lars Ingebrigtsen @ 2019-07-15 20:40 ` Felix E. Klee 2019-07-15 20:48 ` Noam Postavsky ` (2 more replies) 0 siblings, 3 replies; 27+ messages in thread From: Felix E. Klee @ 2019-07-15 20:40 UTC (permalink / raw) To: Lars Ingebrigtsen; +Cc: 32359 On Mon, Jul 15, 2019 at 5:12 PM Lars Ingebrigtsen <larsi@gnus.org> wrote: > The only possible tweak was with using moveto vs. move-to. The latter > looks more Emacsish, but in SVG speak they're moveto (etc), so I think > it makes sense to leave them that way. Good catch. I had the same thought and decided to stick with `moveto`. > Have you gone through the paperwork process with the FSF? I just started with that, following: https://www.gnu.org/prep/maintain/html_node/Copyright-Papers.html A Savannah account I have, and I now requested an fencepost account. Once I have that, I’ll try to find the appropriate template. Sorry, I hope this is no problem. Luckily, there is no issue with copyright assignment: I am 1. old enough (Emacs user for >18y), and 2. not employed, wrote the code for my own pleasure. But I understand that the process can take a while, although nowadays it’s possible to submit the form electronically: “Contributors located in […] Germany […] can print, sign, and then email (or fax) a scanned copy back to the FSF.” ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-07-15 20:40 ` Felix E. Klee @ 2019-07-15 20:48 ` Noam Postavsky 2019-07-16 10:32 ` Felix E. Klee 2019-07-15 21:13 ` Felix E. Klee 2019-07-16 6:05 ` Lars Ingebrigtsen 2 siblings, 1 reply; 27+ messages in thread From: Noam Postavsky @ 2019-07-15 20:48 UTC (permalink / raw) To: Felix E. Klee; +Cc: Lars Ingebrigtsen, 32359 "Felix E. Klee" <felix.klee@inka.de> writes: >> Have you gone through the paperwork process with the FSF? > > I just started with that, following: > > https://www.gnu.org/prep/maintain/html_node/Copyright-Papers.html > > A Savannah account I have, and I now requested an fencepost account. > Once I have that, I’ll try to find the appropriate template. You don't need a Savannah nor fencepost account for this. The form and instructions are at https://git.savannah.gnu.org/cgit/gnulib.git/tree/doc/Copyright/request-assign.future ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-07-15 20:48 ` Noam Postavsky @ 2019-07-16 10:32 ` Felix E. Klee 0 siblings, 0 replies; 27+ messages in thread From: Felix E. Klee @ 2019-07-16 10:32 UTC (permalink / raw) To: Noam Postavsky; +Cc: Lars Ingebrigtsen, 32359 On Tue, Jul 16, 2019 at 3:01 AM Noam Postavsky <npostavs@gmail.com> wrote: > You don't need a Savannah nor fencepost account for this. The form > and instructions are at > https://git.savannah.gnu.org/cgit/gnulib.git/tree/doc/Copyright/request-assign.future I just submitted that form, thanks! (Savannah account would’ve been needed for fencepost, though.) ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-07-15 20:40 ` Felix E. Klee 2019-07-15 20:48 ` Noam Postavsky @ 2019-07-15 21:13 ` Felix E. Klee 2019-07-16 6:05 ` Lars Ingebrigtsen 2 siblings, 0 replies; 27+ messages in thread From: Felix E. Klee @ 2019-07-15 21:13 UTC (permalink / raw) To: Lars Ingebrigtsen; +Cc: 32359 Update: fencepost account got rejected because I’m only a contributor, not a maintainer of a GNU project How do I get access to the necessary copyright papers now? ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-07-15 20:40 ` Felix E. Klee 2019-07-15 20:48 ` Noam Postavsky 2019-07-15 21:13 ` Felix E. Klee @ 2019-07-16 6:05 ` Lars Ingebrigtsen 2019-07-16 10:31 ` Felix E. Klee 2 siblings, 1 reply; 27+ messages in thread From: Lars Ingebrigtsen @ 2019-07-16 6:05 UTC (permalink / raw) To: Felix E. Klee; +Cc: 32359 "Felix E. Klee" <felix.klee@inka.de> writes: > Sorry, I hope this is no problem. Luckily, there is no issue with > copyright assignment: I am 1. old enough (Emacs user for >18y), and > 2. not employed, wrote the code for my own pleasure. But I understand > that the process can take a while, although nowadays it’s possible to > submit the form electronically: Sounds good, but I'll have to revert the patch in the meantime, I think. Should be no problem to re-apply it later when the paperwork is done. -- (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-07-16 6:05 ` Lars Ingebrigtsen @ 2019-07-16 10:31 ` Felix E. Klee 2019-07-16 13:46 ` Lars Ingebrigtsen 0 siblings, 1 reply; 27+ messages in thread From: Felix E. Klee @ 2019-07-16 10:31 UTC (permalink / raw) To: Lars Ingebrigtsen; +Cc: 32359 On Tue, Jul 16, 2019 at 8:05 AM Lars Ingebrigtsen <larsi@gnus.org> wrote: > Sounds good, but I'll have to revert the patch in the meantime, I > think. I understand. It’s no problem at all. I’ll let you know once the paperwork is done. ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-07-16 10:31 ` Felix E. Klee @ 2019-07-16 13:46 ` Lars Ingebrigtsen 2019-07-24 7:49 ` Felix E. Klee 2019-07-30 17:46 ` Felix E. Klee 0 siblings, 2 replies; 27+ messages in thread From: Lars Ingebrigtsen @ 2019-07-16 13:46 UTC (permalink / raw) To: Felix E. Klee; +Cc: 32359 "Felix E. Klee" <felix.klee@inka.de> writes: > On Tue, Jul 16, 2019 at 8:05 AM Lars Ingebrigtsen <larsi@gnus.org> > wrote: >> Sounds good, but I'll have to revert the patch in the meantime, I >> think. > > I understand. It’s no problem at all. I’ll let you know once the > paperwork is done. Great! I've now reverted the patch (and Glenn's follow-up patch that added a menu). Both will be re-reverted once the paperwork's in place. -- (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-07-16 13:46 ` Lars Ingebrigtsen @ 2019-07-24 7:49 ` Felix E. Klee 2019-07-24 14:30 ` Eli Zaretskii 2019-07-30 17:46 ` Felix E. Klee 1 sibling, 1 reply; 27+ messages in thread From: Felix E. Klee @ 2019-07-24 7:49 UTC (permalink / raw) To: Lars Ingebrigtsen; +Cc: 32359 [-- Attachment #1: Type: text/plain, Size: 118 bytes --] Don’t know if anyone can help here, but I sent the attached request (edited) more than a week ago. No reply yet. [-- Attachment #2: Felix Edgar Klee.eml.gz --] [-- Type: application/x-gzip, Size: 690 bytes --] ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-07-24 7:49 ` Felix E. Klee @ 2019-07-24 14:30 ` Eli Zaretskii 0 siblings, 0 replies; 27+ messages in thread From: Eli Zaretskii @ 2019-07-24 14:30 UTC (permalink / raw) To: Felix E. Klee; +Cc: larsi, 32359 > From: "Felix E. Klee" <felix.klee@inka.de> > Date: Wed, 24 Jul 2019 09:49:26 +0200 > Cc: Eli Zaretskii <eliz@gnu.org>, 32359@debbugs.gnu.org > > Don’t know if anyone can help here, but I sent the attached request > (edited) more than a week ago. No reply yet. I suggest to ping them. ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-07-16 13:46 ` Lars Ingebrigtsen 2019-07-24 7:49 ` Felix E. Klee @ 2019-07-30 17:46 ` Felix E. Klee 2019-07-31 20:31 ` Lars Ingebrigtsen 1 sibling, 1 reply; 27+ messages in thread From: Felix E. Klee @ 2019-07-30 17:46 UTC (permalink / raw) To: Lars Ingebrigtsen; +Cc: 32359 Hi Lars, paperwork has been done, i.e. the assignment has been signed by both parties. If you need anything else, please let me know! ―Felix ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-07-30 17:46 ` Felix E. Klee @ 2019-07-31 20:31 ` Lars Ingebrigtsen 2019-08-01 13:23 ` Felix E. Klee 0 siblings, 1 reply; 27+ messages in thread From: Lars Ingebrigtsen @ 2019-07-31 20:31 UTC (permalink / raw) To: Felix E. Klee; +Cc: 32359 "Felix E. Klee" <felix.klee@inka.de> writes: > Hi Lars, > > paperwork has been done, i.e. the assignment has been signed by both > parties. If you need anything else, please let me know! Great! I've now reverted the reverting of the patch, so the svg-path stuff should now be on the Emacs trunk. -- (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-07-31 20:31 ` Lars Ingebrigtsen @ 2019-08-01 13:23 ` Felix E. Klee 2019-08-02 6:40 ` Eli Zaretskii 0 siblings, 1 reply; 27+ messages in thread From: Felix E. Klee @ 2019-08-01 13:23 UTC (permalink / raw) To: Lars Ingebrigtsen; +Cc: 32359 Thanks, Lars! ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-08-01 13:23 ` Felix E. Klee @ 2019-08-02 6:40 ` Eli Zaretskii 2019-08-02 10:43 ` Felix E. Klee 0 siblings, 1 reply; 27+ messages in thread From: Eli Zaretskii @ 2019-08-02 6:40 UTC (permalink / raw) To: Felix E. Klee; +Cc: larsi, 32359 > From: "Felix E. Klee" <felix.klee@inka.de> > Date: Thu, 1 Aug 2019 15:23:36 +0200 > Cc: Eli Zaretskii <eliz@gnu.org>, 32359@debbugs.gnu.org > > Thanks, Lars! Please review the followup change in the documentation I made, as the original had a few nits that needed correcting. In particular, you cannot add a @node without also adding a @menu and mentioning that node in a few other menus. ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-08-02 6:40 ` Eli Zaretskii @ 2019-08-02 10:43 ` Felix E. Klee 2019-08-02 11:57 ` Eli Zaretskii 0 siblings, 1 reply; 27+ messages in thread From: Felix E. Klee @ 2019-08-02 10:43 UTC (permalink / raw) To: Eli Zaretskii; +Cc: Lars Magne Ingebrigtsen, 32359 On Fri, Aug 2, 2019 at 8:40 AM Eli Zaretskii <eliz@gnu.org> wrote: > Please review the followup change in the documentation I made The definition of SVG paths is a good addition IMO. > In particular, you cannot add a @node without also adding a @menu and > mentioning that node in a few other menus. Now that “SVG Path Commands” is a subheading, isn’t the menu superfluous? I find it confusing to the reader. For comparison, have a look in `searching.texi`: @subsubheading Literals There is no menu for “Literals”. ^ permalink raw reply [flat|nested] 27+ messages in thread
* bug#32359: [PATCH] Add svg-path 2019-08-02 10:43 ` Felix E. Klee @ 2019-08-02 11:57 ` Eli Zaretskii 0 siblings, 0 replies; 27+ messages in thread From: Eli Zaretskii @ 2019-08-02 11:57 UTC (permalink / raw) To: Felix E. Klee, Glenn Morris; +Cc: larsi, 32359 > From: "Felix E. Klee" <felix.klee@inka.de> > Date: Fri, 2 Aug 2019 12:43:33 +0200 > Cc: Lars Magne Ingebrigtsen <larsi@gnus.org>, 32359@debbugs.gnu.org, Glenn Morris <rgm@gnu.org> > > Now that “SVG Path Commands” is a subheading, isn’t the menu > superfluous? It should be superfluous. I asked Glenn whether it was really needed; I don't think it is. ^ permalink raw reply [flat|nested] 27+ messages in thread
end of thread, other threads:[~2019-08-02 11:57 UTC | newest] Thread overview: 27+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-08-03 11:07 bug#32359: [PATCH] Add svg-path Felix E. Klee 2018-08-03 13:05 ` Eli Zaretskii 2018-08-23 15:50 ` Felix E. Klee 2018-08-23 17:23 ` Eli Zaretskii 2018-08-24 9:15 ` Felix E. Klee 2019-06-23 22:15 ` Lars Ingebrigtsen 2019-06-24 14:32 ` Felix E. Klee 2019-06-24 14:39 ` Lars Ingebrigtsen 2019-07-12 14:35 ` Felix E. Klee 2019-07-12 16:02 ` Lars Ingebrigtsen 2019-07-15 13:34 ` Felix E. Klee 2019-07-15 15:12 ` Lars Ingebrigtsen 2019-07-15 20:40 ` Felix E. Klee 2019-07-15 20:48 ` Noam Postavsky 2019-07-16 10:32 ` Felix E. Klee 2019-07-15 21:13 ` Felix E. Klee 2019-07-16 6:05 ` Lars Ingebrigtsen 2019-07-16 10:31 ` Felix E. Klee 2019-07-16 13:46 ` Lars Ingebrigtsen 2019-07-24 7:49 ` Felix E. Klee 2019-07-24 14:30 ` Eli Zaretskii 2019-07-30 17:46 ` Felix E. Klee 2019-07-31 20:31 ` Lars Ingebrigtsen 2019-08-01 13:23 ` Felix E. Klee 2019-08-02 6:40 ` Eli Zaretskii 2019-08-02 10:43 ` Felix E. Klee 2019-08-02 11:57 ` Eli Zaretskii
Code repositories for project(s) associated with this external index https://git.savannah.gnu.org/cgit/emacs.git https://git.savannah.gnu.org/cgit/emacs/org-mode.git This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.