Skip to content

Libtommath update to 1.2.0#8917

Closed
Khalyutin wants to merge 18 commits intoFirebirdSQL:masterfrom
notesoft:libtommath-update-to-1.2
Closed

Libtommath update to 1.2.0#8917
Khalyutin wants to merge 18 commits intoFirebirdSQL:masterfrom
notesoft:libtommath-update-to-1.2

Conversation

@Khalyutin
Copy link
Contributor

@AlexPeshkoff AlexPeshkoff self-assigned this Feb 25, 2026
inc_file_name[ii] = line[i];
}
inc_file_name[ii] = 0;

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How is it related with libtommath update? :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If possible, of course, please pay attention for:

#8887
#8893

are pending. We are preparing about ten more corrections... (not fount #include files, c++20 standard error etc.)
It would be much easier for us to debug and testing...

@@ -3121,6 +3121,8 @@ static bool packet_receive(rem_port* port, UCHAR* buffer, SSHORT buffer_length,
INET_force_error = 1;
try
{
// function inet_error as 5 parameters
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please do not put all stuff into single PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR was approved earlier.
#8882..
I don't understand why this change was repeated here...
and how to remove it from here :(

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For example create new fresh branch and cherry-pick required commits from this branch.

@hvlad
Copy link
Member

hvlad commented Feb 25, 2026

Why 1.2.0 ? AFAIU, latest stable release is 1.3.0

@Khalyutin
Copy link
Contributor Author

Why 1.2.0 ? AFAIU, latest stable release is 1.3.0

in the Linux dockerfile we noticed downloads and compilations on the fly libtommath v1.2.0.

  • We didn't want to add different versions of the library.
  • Besides, my experience tells me it's safer to switch to the second-to-last version when a new one is released. :)

Copy link
Contributor Author

@Khalyutin Khalyutin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AlexPeshkoff
Copy link
Member

Why 1.2.0 ? AFAIU, latest stable release is 1.3.0

in the Linux dockerfile we noticed downloads and compilations on the fly libtommath v1.2.0.

Prebuilt linux binaries can use any version of library which is installed in host OS. Yes, docker is using 1.2, but it makes no difference for the rest of platforms.

* Besides, my experience tells me it's safer to switch to the second-to-last version when a new one is released. :)

Gentoo stable (known with it's careful approach of choosing really stable versions) is using 1.3.0. I agree with Vlad - let's upgrade to it.

@Khalyutin
Copy link
Contributor Author

Okay, but we need to test again for 1.3 ...

@Khalyutin Khalyutin closed this Feb 26, 2026
@Khalyutin
Copy link
Contributor Author

Libtommath update to 1.3.0
#8919

@Khalyutin Khalyutin deleted the libtommath-update-to-1.2 branch February 26, 2026 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants