[Geany-devel] Geany Plugins SVN Commit URLs

Matthew Brush mbrush at xxxxx
Wed Nov 2 00:01:21 UTC 2011

On 11/01/2011 07:50 AM, Colomban Wendling wrote:
> Le 01/11/2011 15:43, Matthew Brush a écrit :
>> On 11-11-01 06:11 AM, Jiří Techet wrote:
>>> On Sun, Oct 30, 2011 at 23:25, Matthew Brush<mbrush at codebrainz.ca>
>>> wrote:
>>>> On 11-10-30 03:03 PM, Frank Lanitz wrote:
>>>>> On Sun, 30 Oct 2011 02:15:33 -0700
>>>>> Matthew Brush<mbrush at codebrainz.ca>     wrote:
>>>>>> Hi,
>>>>>> I haven't heard too much about the Git conversion of Geany Plugins,
>>>>>> but I wanted to offer a suggestion that might improve the commit
>>>>>> messages compared to Geany's.  In the Geany's Git repository there's
>>>>>> links back to the SVN repository in the commit messages (the links
>>>>>> are broken BTW). I was wondering if it would be better to have these
>>>>>> links point to a place we have more control over, for example using a
>>>>>> Geany commit, instead of this in the commit message:
>>>>>> git-svn-id:
>>>>>> https://geany.svn.sourceforge.net/svnroot/geany/trunk@5970
>>>>>> ea778897-0a13-0410-b9d1-a72fbfd435f5
>>>>>> We could have something like this:
>>>>>> git-svn-id:
>>>>>> [ea778897-0a13-0410-b9d1-a72fbfd435f5]
>>>>>> (http://svn.geany.org/svnroot/geany/trunk?rev=5970)
>>>>>> That way we could make the links work and be able to move the old SVN
>>>>>> repository without breaking all the links in the commit message. I
>>>>>> guess we'd need a little redirect script or something on geany.org.
>>>>>> Note also the use of Markdown link syntax, which might render better
>>>>>> on Github (not sure).
>>>>> I'm afraid I don't get what is needed here? Did I got you right to
>>>>> setup complete mirror of ancient svn on geany.org?
>>>> Or easier, just make some URL forwarders/rewrites on geany.org so we
>>>> don't
>>>> have to depend on SourceForge.net to always store the SVN repo and never
>>>> change URLs.  The SVN URLs in the Git commit messages will never be
>>>> able to
>>>> change ever in the future, so if we point it to a location we
>>>> control, then
>>>> we can ensure they remain valid.
>>>>   From Jiri's other message though, it sounds like the SVN URLs in the
>>>> Git
>>>> commit messages are all going to be broken like the one's in Geany's Git
>>>> repository, since it's not an option of the conversion script to change
>>>> them.  I don't feel like hacking the conversion script to change the URL
>>>> that goes in the commit message, and anyway, as long as it contains
>>>> the SVN
>>>> revision, it's still useful.
>>> Another option would be to use
>>> git filter-branch --msg-filter command
>>> where command would be some sed expression which would substitute the
>>> current format with the required format. Maybe a bit easier, still
>>> something I don't want to spend my time on ;-). The only reason I
>>> included the SVN metadata was the commit ID which is useful to have
>>> because some commits refer to it in their comments and this
>>> information would be lost otherwise.
>> What about something as simple as:
>> sed |git-svn-id:
>> https://geany-plugins.svn.sourceforge.net/svnroot/geany-plugins/trunk/geany-plugins|SVN:|
>> To at least remove the invalid URL, or better yet:
>> sed |geany-plugins@|geany-plugins?revision=|
>> Or similar, so that the URL is not broken?
>> If it's easy enough, I'd say it's worthwhile.
> Although I don't mind much, isn't "@rev" more readable than
> "?revision=rev"?  I mean, the history will be in Git already, so is it
> more important to have a valid URL to the old SVN repo or a readable
> reference?

IMO, if there's going to be a hyperlink, it should not be broken, 
otherwise we should use something like the first option, to just remove 
the broken URL all together and have something like `SVN revision 1234` 
or `SVN @1234` or something.

Matthew Brush

More information about the Devel mailing list