MantisBT - opensim
View Issue Details
0007479opensim[GRID] Asset Servicepublic2015-03-04 09:032015-03-04 11:06
cinderblocks 
BlueWall 
immediatemajorhave not tried
closedfixed 
master (dev code) 
master (dev code) 
Grid (1 Region per Sim)
BulletSim
Mono / Linux64
Other
0007479: XAssetStore DB needs critical db migration change
This morning I realized that after both my patch to extend the asset description length to 128 does not cover XAssetStore's db tables.

Attaching a patch.
Note that this patch covers MySQL and PGSql, but not SQLite because it appears there is no sqlite migration file for XAssetStore
No tags attached.
diff XAssetStore-patch.diff (2,942) 2015-03-04 10:12
http://opensimulator.org/mantis/file_download.php?file_id=4121&type=bug
Issue History
2015-03-04 09:03cinderblocksNew Issue
2015-03-04 09:03cinderblocksFile Added: XAssetStore-patch.diff
2015-03-04 09:05cinderblocksStatusnew => patch included
2015-03-04 09:56cinderblocksFile Deleted: XAssetStore-patch.diff
2015-03-04 09:57cinderblocksFile Added: XAssetStore-patch.diff
2015-03-04 10:11cinderblocksFile Deleted: XAssetStore-patch.diff
2015-03-04 10:12cinderblocksFile Added: XAssetStore-patch.diff
2015-03-04 10:12cinderblocksNote Added: 0027735
2015-03-04 10:25BlueWallAssigned To => BlueWall
2015-03-04 10:25BlueWallStatuspatch included => assigned
2015-03-04 10:45BlueWallNote Added: 0027737
2015-03-04 10:47BlueWallNote Added: 0027738
2015-03-04 11:04BlueWallNote Edited: 0027738bug_revision_view_page.php?bugnote_id=27738#r3804
2015-03-04 11:05BlueWallNote Added: 0027740
2015-03-04 11:06BlueWallMono Versiontrunk => Other
2015-03-04 11:06BlueWallNote Added: 0027741
2015-03-04 11:06BlueWallStatusassigned => resolved
2015-03-04 11:06BlueWallFixed in Version => master (dev code)
2015-03-04 11:06BlueWallResolutionopen => fixed
2015-03-04 11:06BlueWallStatusresolved => closed

Notes
(0027735)
cinderblocks   
2015-03-04 10:12   
Trying again with the patch. Someone with postgresql please test. :)
(0027737)
BlueWall   
2015-03-04 10:45   
This is failing, maybe because of the earlier patch that was applied?
(0027738)
BlueWall   
2015-03-04 10:47   
(edited on: 2015-03-04 11:04)
OK, reverted the previous patch and it applies cleanly now. Thanks.

(0027740)
BlueWall   
2015-03-04 11:05   
Applied. Thanks for the patches!

PS: In the future, just leave the 1st patch and make the 2nd based on the changes from there.
(0027741)
BlueWall   
2015-03-04 11:06   
Thanks for the patches!