Release file sql-wasm.js doesn't support JSON1 after rebuilding, but debug one does.

Ouverte
#442 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
3/5
Temps estimé
1-2 jours
Accessibilité débutants
45/100
Type d'issue
Bug
Clarté
Plutôt claire
Activité
À l'abandon
Stack technique
javascript, sqlite, wasm
Domaine
build-system, databases

Piste de recherche

Start with the Makefile CFLAGS and the npm run rebuild entry point, then compare how sql-wasm.js and sql-wasm-debug.js are produced in dist. Reproduce the JSON1 test against both outputs and trace the release build configuration; done means the rebuilt release file supports JSON1 as the debug file does.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

Hi,
I clone the project and did the modification in makefile to support JSON1.

CFLAGS = \
	-O2 \
	-DSQLITE_OMIT_LOAD_EXTENSION \
	-DSQLITE_DISABLE_LFS \
	-DSQLITE_ENABLE_FTS3 \
	-DSQLITE_ENABLE_FTS3_PARENTHESIS \
	-DSQLITE_ENABLE_FTS5 \
    	-DSQLITE_ENABLE_JSON1 \
	-DSQLITE_THREADSAFE=0 \
	-DSQLITE_ENABLE_NORMALIZE

Then rebuild

npm run rebuild

The output files in dist folder, I found that only debug one (sql-wasm-debug.js) supports JSON1 by running some test code. But the release file sql-wasm.js doesn't support JSON1.

Is there any idea how to resolve this issue? Thank you

Langage dominant
JavaScript
Étoiles
13.7k
Forks
1.1k
Aucune PR mergée en 30 j

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de sql-js/sql.js

Toutes les issues de sql-js/sql.js

Issues similaires

Plus d'issues JavaScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.