ericsink / SQLitePCL.raw

A Portable Class Library (PCL) for low-level (raw) access to SQLite
Apache License 2.0
525 stars 109 forks source link

UWP Sqlite 'VACUUM; error #339

Closed ericsink closed 2 years ago

ericsink commented 4 years ago

Investigate dotnet/efcore#20253

ericsink commented 4 years ago

@fernand-delavy @bricelam

I think the problem is that sqlite3_win32_set_directory() is never being called.

This was one of the breaking changes in SQLitePCLRaw 2.x:

https://github.com/ericsink/SQLitePCL.raw/blob/master/v2.md

I took the Test.zip file posted as part of the original issue, removed Microsoft.Data.Sqlite, added SQLitePCLRaw 2.0.2, and tweaked the code accordingly. This reproduces the same error mentioned by the original issue. Then I added sqlite3_win32_set_directory() calls, and the error goes away.

Fernand-Delavy commented 4 years ago

Hello Sir,

I'd appreciate it if you send back the modified Test App, thank you

Regards

Fernand Delavy Rue Carlo Hemmerling 11 CH-1804 Corsier-sur-Vevey Email : fernand.delavy@outlook.com Tél : +41 21 922 85 65 Mobile : +41 79 668 29 94

De : Eric Sink notifications@github.com Envoyé : mercredi, 18 mars 2020 13:39 À : ericsink/SQLitePCL.raw SQLitePCL.raw@noreply.github.com Cc : Fernand-Delavy fernand.delavy@outlook.com; Mention mention@noreply.github.com Objet : Re: [ericsink/SQLitePCL.raw] UWP Sqlite 'VACUUM; error (#339)

@Fernand-Delavyhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FFernand-Delavy&data=02%7C01%7C%7C97d983aa2efe4902c2a008d7cb395ee2%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637201319564163180&sdata=Vk6gTXZrLQhV49og3wUVgzv7XoaD2cmr%2BhU1cKbTA14%3D&reserved=0 @bricelamhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbricelam&data=02%7C01%7C%7C97d983aa2efe4902c2a008d7cb395ee2%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637201319564173188&sdata=ykiUT5xqdnWa4cY5ajkLYcRTZ06OVqULFZlBzDE0il4%3D&reserved=0

I think the problem is that sqlite3_win32_set_directory() is never being called.

This was one of the breaking changes in SQLitePCLRaw 2.x:

https://github.com/ericsink/SQLitePCL.raw/blob/master/v2.mdhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fericsink%2FSQLitePCL.raw%2Fblob%2Fmaster%2Fv2.md&data=02%7C01%7C%7C97d983aa2efe4902c2a008d7cb395ee2%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637201319564173188&sdata=u9FL6exRkrLvAKJZZyEpL9RfVOU1qflJfKhXFG7VX7M%3D&reserved=0

I took the Test.zip file posted as part of the original issue, removed Microsoft.Data.Sqlite, added SQLitePCLRaw 2.0.2, and tweaked the code accordingly. This reproduces the same error mentioned by the original issue. Then I added sqlite3_win32_set_directory() calls, and the error goes away.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fericsink%2FSQLitePCL.raw%2Fissues%2F339%23issuecomment-600599509&data=02%7C01%7C%7C97d983aa2efe4902c2a008d7cb395ee2%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637201319564183196&sdata=4Y2Kck3wbSu3TSf32U0rdilo9MSJx46ye3HKlFlcsw8%3D&reserved=0, or unsubscribehttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKKWCWLGG5KDU7ZAVGU27ODRIC6HHANCNFSM4LHEHNEQ&data=02%7C01%7C%7C97d983aa2efe4902c2a008d7cb395ee2%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637201319564193200&sdata=2RwTdaQ3ycpuVcYD8%2F6t7g3PfIRBtaPX6O8gQowJk6o%3D&reserved=0.

ericsink commented 4 years ago

Test2.zip

ericsink commented 2 years ago

Closing old/stale issue.