Stuart Henderson <s...@spacehopper.org> writes:

> On 2021/07/14 18:34, Omar Polo wrote:
>> Hello,
>> 
>> Straightforward update for net/tdlib from 1.7.3 to 1.7.5.  telega.el
>> (not in ports) works, and net/tg is happy too.
>
> Thanks, committed -
>
>> I got the following error while trying to update the port, thought.
>> Probably was something strange on my end, since I had the package tdlib
>> installed locally for a long time before it went into ports, but I'm
>> mentioning it anyway.  It seemed like the port infrastructure treated
>> tdlib-1.7.3 and tdlib-1.7.5 as different packages:
>> 
>> ----8<----
>> % make update
>> ===> Updating for tdlib-1.7.5
>> Not installed, no update
>> % make install
>> ===>  Verifying specs: c++ c++abi pthread crypto m ssl z
>> ===>  found c++.8.0 c++abi.5.0 pthread.26.1 crypto.46.3 m.10.1 ssl.49.0 z.6.0
>> ===>  Installing tdlib-1.7.5 from /usr/ports/packages/amd64/all/
>> Can't install tdlib-1.7.5 because of conflicts (tdlib-1.7.3)
>
> That's likely a differing pkgpath from previous test builds,
> if you still have the old package around check the @pkgpath line in
> +CONTENTS.

Thanks for the pointer! :)

>> --- tdlib-1.7.5 -------------------
>> Can't install tdlib-1.7.5: conflicts
>> Couldn't install tdlib-1.7.5
>> *** Error 1 in . (/usr/ports/infrastructure/mk/bsd.port.mk:2159 
>> '/var/db/pkg/tdlib-1.7.5/+CONTENTS': @/usr/bin/env -i PKG_TMPDIR=/var/tmp 
>> TE...)
>> *** Error 2 in /usr/ports/net/tdlib 
>> (/usr/ports/infrastructure/mk/bsd.port.mk:2602 'install': @lock=tdlib-1.7.5; 
>>  export _LOCKS_HELD=" tdlib...)
>> ---->8----
>> 
>> Oh, and another thing.  I haven't bumped the solib version, but I'd like
>> if another pair of eyes could verify I'm not mistaking anything.  Below
>> the diff of `nm -D' before and after the update.
>
> There's a new exported function td_set_log_message_callback so a minor
> bump is warranted.

I'm ashamed, I haven't spotted td_set_log_message_callback...  Thanks
for checking!

> A major bump would be needed if there's either a removal or a change in
> number/order/types of function arguments, or layout of some structs.
> I started to look for these (diffing headers is usually the way)
> but a lot of comments were added and it got rather hard to read so
> I may have missed something, though it's relatively unlikely in a
> minor update.

Thanks,


Omar Polo

Reply via email to