diff options
author | Michael Wood <michael.g.wood@intel.com> | 2016-07-21 14:43:30 +0100 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2016-07-26 08:10:36 +0100 |
commit | 97278fb51c3d4fe46c2a3110015e291fdc5d502d (patch) | |
tree | 2535f7584e2ccc4390ecfbaf16031e8b8886e4c5 /bitbake/lib/toaster/toastergui | |
parent | 8b3146007f98ba6618662cc47ed1c1491ef8945a (diff) | |
download | poky-97278fb51c3d4fe46c2a3110015e291fdc5d502d.tar.gz |
bitbake: toaster: orm Remove the layerindex specific up_branch fields
We don't need to keep track of layerindex data in our database. And
using branch==release is very confusing in the schema. Instead use the
existing Release definition to keep track of which release a
layer_version is for.
Remove the Branch model and all references to it.
Create a migration path to convert from up_branches to their
corresponding releases.
(Bitbake rev: f8f4cffe6fd371f3a7e63690c68f3fcb5dc1f297)
Signed-off-by: Michael Wood <michael.g.wood@intel.com>
Signed-off-by: Elliot Smith <elliot.smith@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'bitbake/lib/toaster/toastergui')
-rw-r--r-- | bitbake/lib/toaster/toastergui/fixtures/toastergui-unittest-data.xml | 21 | ||||
-rw-r--r-- | bitbake/lib/toaster/toastergui/tests.py | 2 | ||||
-rwxr-xr-x | bitbake/lib/toaster/toastergui/views.py | 8 |
3 files changed, 17 insertions, 14 deletions
diff --git a/bitbake/lib/toaster/toastergui/fixtures/toastergui-unittest-data.xml b/bitbake/lib/toaster/toastergui/fixtures/toastergui-unittest-data.xml index 80c1e19dda..4517ed1765 100644 --- a/bitbake/lib/toaster/toastergui/fixtures/toastergui-unittest-data.xml +++ b/bitbake/lib/toaster/toastergui/fixtures/toastergui-unittest-data.xml | |||
@@ -1,5 +1,16 @@ | |||
1 | <?xml version="1.0" encoding="utf-8"?> | 1 | <?xml version="1.0" encoding="utf-8"?> |
2 | <django-objects version="1.0"> | 2 | <django-objects version="1.0"> |
3 | <object pk="1" model="orm.bitbakeversion"> | ||
4 | <field type="CharField" name="name">v2.3</field> | ||
5 | <field type="GitURLField" name="giturl">git://git.openembedded.org/bitbake</field> | ||
6 | <field type="CharField" name="dirpath">b</field> | ||
7 | <field type="CharField" name="branch">a</field> | ||
8 | </object> | ||
9 | <object pk="1" model="orm.release"> | ||
10 | <field type="CharField" name="name">master</field> | ||
11 | <field type="CharField" name="description">master project</field> | ||
12 | <field to="orm.bitbake_version" name="bitbake_version">1</field> | ||
13 | </object> | ||
3 | <object pk="1" model="orm.project"> | 14 | <object pk="1" model="orm.project"> |
4 | <field type="CharField" name="name">a test project</field> | 15 | <field type="CharField" name="name">a test project</field> |
5 | <field type="CharField" name="short_description"></field> | 16 | <field type="CharField" name="short_description"></field> |
@@ -332,10 +343,6 @@ | |||
332 | <field type="CharField" name="branch_name">master</field> | 343 | <field type="CharField" name="branch_name">master</field> |
333 | <field type="TextField" name="helptext"><None></None></field> | 344 | <field type="TextField" name="helptext"><None></None></field> |
334 | </object> | 345 | </object> |
335 | <object pk="1" model="orm.branch"> | ||
336 | <field type="CharField" name="name">master</field> | ||
337 | <field type="CharField" name="short_description"></field> | ||
338 | </object> | ||
339 | <object pk="1" model="orm.layer"> | 346 | <object pk="1" model="orm.layer"> |
340 | <field type="DateTimeField" name="up_date"><None></None></field> | 347 | <field type="DateTimeField" name="up_date"><None></None></field> |
341 | <field type="CharField" name="name">a layer</field> | 348 | <field type="CharField" name="name">a layer</field> |
@@ -351,7 +358,7 @@ | |||
351 | <field to="orm.build" name="build" rel="ManyToOneRel">1</field> | 358 | <field to="orm.build" name="build" rel="ManyToOneRel">1</field> |
352 | <field to="orm.layer" name="layer" rel="ManyToOneRel">1</field> | 359 | <field to="orm.layer" name="layer" rel="ManyToOneRel">1</field> |
353 | <field type="DateTimeField" name="up_date"><None></None></field> | 360 | <field type="DateTimeField" name="up_date"><None></None></field> |
354 | <field to="orm.branch" name="up_branch" rel="ManyToOneRel">1</field> | 361 | <field to="orm.release" name="release" rel="ManyToOneRel">1</field> |
355 | <field type="CharField" name="branch">master</field> | 362 | <field type="CharField" name="branch">master</field> |
356 | <field type="CharField" name="commit">abcdef123</field> | 363 | <field type="CharField" name="commit">abcdef123</field> |
357 | <field type="CharField" name="dirpath">/tmp/</field> | 364 | <field type="CharField" name="dirpath">/tmp/</field> |
@@ -363,7 +370,7 @@ | |||
363 | <field to="orm.build" name="build" rel="ManyToOneRel"><None></None></field> | 370 | <field to="orm.build" name="build" rel="ManyToOneRel"><None></None></field> |
364 | <field to="orm.layer" name="layer" rel="ManyToOneRel">2</field> | 371 | <field to="orm.layer" name="layer" rel="ManyToOneRel">2</field> |
365 | <field type="DateTimeField" name="up_date"><None></None></field> | 372 | <field type="DateTimeField" name="up_date"><None></None></field> |
366 | <field to="orm.branch" name="up_branch" rel="ManyToOneRel">1</field> | 373 | <field to="orm.release" name="release" rel="ManyToOneRel">1</field> |
367 | <field type="CharField" name="branch">testing-branch</field> | 374 | <field type="CharField" name="branch">testing-branch</field> |
368 | <field type="CharField" name="commit">9876fedcba</field> | 375 | <field type="CharField" name="commit">9876fedcba</field> |
369 | <field type="CharField" name="dirpath"><None></None></field> | 376 | <field type="CharField" name="dirpath"><None></None></field> |
@@ -375,7 +382,7 @@ | |||
375 | <field to="orm.build" name="build" rel="ManyToOneRel">1</field> | 382 | <field to="orm.build" name="build" rel="ManyToOneRel">1</field> |
376 | <field to="orm.layer" name="layer" rel="ManyToOneRel">2</field> | 383 | <field to="orm.layer" name="layer" rel="ManyToOneRel">2</field> |
377 | <field type="DateTimeField" name="up_date"><None></None></field> | 384 | <field type="DateTimeField" name="up_date"><None></None></field> |
378 | <field to="orm.branch" name="up_branch" rel="ManyToOneRel">1</field> | 385 | <field to="orm.release" name="release" rel="ManyToOneRel">1</field> |
379 | <field type="CharField" name="branch">testing-branch</field> | 386 | <field type="CharField" name="branch">testing-branch</field> |
380 | <field type="CharField" name="commit">9876fedcba</field> | 387 | <field type="CharField" name="commit">9876fedcba</field> |
381 | <field type="CharField" name="dirpath"><None></None></field> | 388 | <field type="CharField" name="dirpath"><None></None></field> |
diff --git a/bitbake/lib/toaster/toastergui/tests.py b/bitbake/lib/toaster/toastergui/tests.py index 03b73a9d69..61ac4773f3 100644 --- a/bitbake/lib/toaster/toastergui/tests.py +++ b/bitbake/lib/toaster/toastergui/tests.py | |||
@@ -31,7 +31,7 @@ from orm.models import Project, Release, BitbakeVersion, Package, LogMessage | |||
31 | from orm.models import LayerSource, Layer, Build | 31 | from orm.models import LayerSource, Layer, Build |
32 | from orm.models import Layer_Version, Recipe, Machine, ProjectLayer, Target | 32 | from orm.models import Layer_Version, Recipe, Machine, ProjectLayer, Target |
33 | from orm.models import CustomImageRecipe, ProjectVariable | 33 | from orm.models import CustomImageRecipe, ProjectVariable |
34 | from orm.models import Branch, CustomImagePackage | 34 | from orm.models import CustomImagePackage |
35 | 35 | ||
36 | import toastermain | 36 | import toastermain |
37 | import inspect | 37 | import inspect |
diff --git a/bitbake/lib/toaster/toastergui/views.py b/bitbake/lib/toaster/toastergui/views.py index d19815f661..a40ceef942 100755 --- a/bitbake/lib/toaster/toastergui/views.py +++ b/bitbake/lib/toaster/toastergui/views.py | |||
@@ -161,7 +161,7 @@ def _lv_to_dict(prj, x = None): | |||
161 | return {"id": x.pk, | 161 | return {"id": x.pk, |
162 | "name": x.layer.name, | 162 | "name": x.layer.name, |
163 | "tooltip": "%s | %s" % (x.layer.vcs_url,x.get_vcs_reference()), | 163 | "tooltip": "%s | %s" % (x.layer.vcs_url,x.get_vcs_reference()), |
164 | "detail": "(%s" % x.layer.vcs_url + (")" if x.up_branch == None else " | "+x.get_vcs_reference()+")"), | 164 | "detail": "(%s" % x.layer.vcs_url + (")" if x.release == None else " | "+x.get_vcs_reference()+")"), |
165 | "giturl": x.layer.vcs_url, | 165 | "giturl": x.layer.vcs_url, |
166 | "layerdetailurl" : reverse('layerdetails', args=(prj.id,x.pk)), | 166 | "layerdetailurl" : reverse('layerdetails', args=(prj.id,x.pk)), |
167 | "revision" : x.get_vcs_reference(), | 167 | "revision" : x.get_vcs_reference(), |
@@ -1288,7 +1288,7 @@ if True: | |||
1288 | from django.contrib.auth import authenticate, login | 1288 | from django.contrib.auth import authenticate, login |
1289 | from django.contrib.auth.decorators import login_required | 1289 | from django.contrib.auth.decorators import login_required |
1290 | 1290 | ||
1291 | from orm.models import Branch, LayerSource, ToasterSetting, Release, Machine, LayerVersionDependency | 1291 | from orm.models import LayerSource, ToasterSetting, Release, Machine, LayerVersionDependency |
1292 | from bldcontrol.models import BuildRequest | 1292 | from bldcontrol.models import BuildRequest |
1293 | 1293 | ||
1294 | import traceback | 1294 | import traceback |
@@ -1667,9 +1667,6 @@ if True: | |||
1667 | post_data[key] = val.strip() | 1667 | post_data[key] = val.strip() |
1668 | 1668 | ||
1669 | 1669 | ||
1670 | # We need to know what release the current project is so that we | ||
1671 | # can set the imported layer's up_branch_id | ||
1672 | |||
1673 | try: | 1670 | try: |
1674 | layer, layer_created = Layer.objects.get_or_create(name=post_data['name']) | 1671 | layer, layer_created = Layer.objects.get_or_create(name=post_data['name']) |
1675 | except MultipleObjectsReturned: | 1672 | except MultipleObjectsReturned: |
@@ -1759,7 +1756,6 @@ if True: | |||
1759 | 1756 | ||
1760 | return HttpResponse(jsonfilter(json_response), content_type = "application/json") | 1757 | return HttpResponse(jsonfilter(json_response), content_type = "application/json") |
1761 | 1758 | ||
1762 | |||
1763 | @xhr_response | 1759 | @xhr_response |
1764 | def xhr_customrecipe(request): | 1760 | def xhr_customrecipe(request): |
1765 | """ | 1761 | """ |