Re: [PATCH] add an option to htags for specifing "?view=" in viewcvs URL

[Shigio YAMAGUCHI]

Hi,

The revised patch works well. I have committed.

The default view mode of Viewcvs in Savannah seems to have

been changed from 'log' to 'co'. The latest Cvs migration seems

to have affected.

http://lists.gnu.org/archive/html/savannah-hackers-public/2016-12/msg00014.html

Since it may be a mistake, I have sent a report to the support group.

In any case, however, we do not have to worry anymore. Thank you.

Regards,

Shigio

2016-12-17 4:26 GMT+09:00 Hideki IWAMOTO <address@hidden>:

Viewcvs version 0.9 and freebsd cvsweb simply ignore unknown parameters.
Therefore there was no problem adding "?view=log" unconditionally, and
adding a new option is unnecessary.
Revised patch is attached.

 htags/src2html.c |    4 +++-
 1 files changed, 3 insertions(+), 1 deletion(-)

On Fri, 16 Dec 2016 23:39:25 +0900
Hideki IWAMOTO <address@hidden> wrote:

> Hi.
>
> The attached patch adds an option to htags for specifing the value of parameter "view"
> in viewcvs URL.
> "?view=log" is required when accessing the viewvc server that default_file_view
> configuration option is not set to "log".
>
>
> diffstat:
>  htags/htags.c    |    6 ++++++
>  htags/htags.h    |    1 +
>  htags/manual.in  |    2 ++
>  htags/src2html.c |    4 ++++
>  4 files changed, 13 insertions(+)
>
> --
> Hideki IWAMOTO <address@hidden>

--
Hideki IWAMOTO <address@hidden>

_______________________________________________
Bug-global mailing list
address@hidden
https://lists.gnu.org/mailman/listinfo/bug-global

--

Shigio YAMAGUCHI <address@hidden>

PGP fingerprint: D1CB 0B89 B346 4AB6 5663  C4B6 3CA5 BBB3 57BE DDA3