[FFmpeg-devel] [PATCH] avformat/http: include version.h

[FFmpeg-devel] [PATCH] avformat/http: include version.h

[James Almer]

This is needed to get LIBAVFORMAT_VERSION, used as part of the user agent.
Fixes a recent regression.

Signed-off-by: James Almer <jamrial@gmail.com>
---
I dislike the fact AV_STRINGIFY() silenced the warning that should have been
printed about LIBAVFORMAT_VERSION being undefined.

Before:
./ffmpeg -h protocol=http
-user_agent        <string>     .D......... override User-Agent header (default "Lavf/LIBAVFORMAT_VERSION")

After:
./ffmpeg -h protocol=http
-user_agent        <string>     .D......... override User-Agent header (default "Lavf/59.25.100")

I suppose this is a regression since the recent work to split the library
version defines into separate headers.

 libavformat/http.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/libavformat/http.c b/libavformat/http.c
index f80ea7bf35..c5c48c7900 100644
--- a/libavformat/http.c
+++ b/libavformat/http.c
@@ -41,6 +41,7 @@
 #include "network.h"
 #include "os_support.h"
 #include "url.h"
+#include "version.h"
 
 /* XXX: POST protocol is not completely implemented because ffmpeg uses
  * only a subset of it. */
-- 
2.36.1

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avformat/http: include version.h

[Andreas Rheinhardt]

James Almer:
> This is needed to get LIBAVFORMAT_VERSION, used as part of the user agent.
> Fixes a recent regression.
> 
> Signed-off-by: James Almer <jamrial@gmail.com>
> ---
> I dislike the fact AV_STRINGIFY() silenced the warning that should have been
> printed about LIBAVFORMAT_VERSION being undefined.
> 
> Before:
> ./ffmpeg -h protocol=http
> -user_agent        <string>     .D......... override User-Agent header (default "Lavf/LIBAVFORMAT_VERSION")
> 
> After:
> ./ffmpeg -h protocol=http
> -user_agent        <string>     .D......... override User-Agent header (default "Lavf/59.25.100")
> 
> I suppose this is a regression since the recent work to split the library
> version defines into separate headers.
> 
>  libavformat/http.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/libavformat/http.c b/libavformat/http.c
> index f80ea7bf35..c5c48c7900 100644
> --- a/libavformat/http.c
> +++ b/libavformat/http.c
> @@ -41,6 +41,7 @@
>  #include "network.h"
>  #include "os_support.h"
>  #include "url.h"
> +#include "version.h"
>  
>  /* XXX: POST protocol is not completely implemented because ffmpeg uses
>   * only a subset of it. */

LGTM.

- Andreas
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avformat/http: include version.h

[James Almer]

On 6/24/2022 11:06 PM, Andreas Rheinhardt wrote:
> James Almer:
>> This is needed to get LIBAVFORMAT_VERSION, used as part of the user agent.
>> Fixes a recent regression.
>>
>> Signed-off-by: James Almer <jamrial@gmail.com>
>> ---
>> I dislike the fact AV_STRINGIFY() silenced the warning that should have been
>> printed about LIBAVFORMAT_VERSION being undefined.
>>
>> Before:
>> ./ffmpeg -h protocol=http
>> -user_agent        <string>     .D......... override User-Agent header (default "Lavf/LIBAVFORMAT_VERSION")
>>
>> After:
>> ./ffmpeg -h protocol=http
>> -user_agent        <string>     .D......... override User-Agent header (default "Lavf/59.25.100")
>>
>> I suppose this is a regression since the recent work to split the library
>> version defines into separate headers.
>>
>>   libavformat/http.c | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/libavformat/http.c b/libavformat/http.c
>> index f80ea7bf35..c5c48c7900 100644
>> --- a/libavformat/http.c
>> +++ b/libavformat/http.c
>> @@ -41,6 +41,7 @@
>>   #include "network.h"
>>   #include "os_support.h"
>>   #include "url.h"
>> +#include "version.h"
>>   
>>   /* XXX: POST protocol is not completely implemented because ffmpeg uses
>>    * only a subset of it. */
> 
> LGTM.
> 
> - Andreas

Applied, thanks.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".