Jezz

Sure, the textures would be fine once xbt'd. But my concern is that if
case-sensitivity wasn't taken in cosideration for textures, it may not
have been taken into consideration elsewhere either. As an example,
Includes.xml -> includes.xml broke linux entirely. I would hate to
push this thing into the repo without giving the author a chance to
see how differently it may behave in an environment other than the one
he's been using for dev.

Regards,
Cory

On Sat, Apr 21, 2012 at 6:15 AM, Jezz_X <[email protected]> wrote:
> Well obviously the best way is to do it from the start :) but other
> than that I use Notepad++ with the "Find in files" option and then
> have explorer open in the media dir and search for the file name in
> all xml files case insensitive which will show you all times its
> reference in all files and how you typed it you can then double click
> and edit each wrong one.  Sure its a shitty slow manual method but it
> is a do the hard work once and forget about it thing :) I am also sure
> that a more clever person could do a easy regex find a replace as well
> but that's too crazy for me :)
>
> As for the Default..... images best too match confluences case since
> they were changed a long time ago to match the c++ code calls
> https://github.com/xbmc/xbmc/tree/master/addons/skin.confluence/media
>
> On Sat, Apr 21, 2012 at 8:04 PM, Nessus <[email protected]> wrote:
>> I see. Thanks for the info Jezz_X.
>> I did a quick check to skins file names and all of them are case sensitive
>> except genres files, studios files and the default icons (DefaultMovies.png,
>> DefaultAudio.png. etc).
>> Is there any batch procedure to check for case sensitive issues the code or
>> do I have to do it one by one ?.
>>
>> Thanks
>> Nessus
>>
>> --------------------------------------------------
>> From: "Jezz_X" <[email protected]>
>> Sent: Saturday, April 21, 2012 12:45 PM
>> To: "Nessus" <[email protected]>
>> Cc: <[email protected]>
>> Subject: Re: [Xbmc-addons] [Git Pull] Skin Xperience More
>>
>>> It is indeed the problem but again is not really a problem once the
>>> skin is built
>>> Basically everything but Win32 is case sensitive so if the file name
>>> case is not exactly the same as the case in all the xml files it wont
>>> find them.  But on the other hand once it gets compressed into the xbt
>>> then it becomes un-case sensitive and I dare say TheUni is using the
>>> raw images as the test before he commits it.
>>> Even though as I said its not a "real" issue it is best practices to
>>> fix it and do it correctly
>>>
>>> On Sat, Apr 21, 2012 at 7:37 PM, Nessus <[email protected]> wrote:
>>>> Unfortunately not at the moment but I can setup a XBMCbuntu installation
>>>> to
>>>> an extra HDisk on my windows machine.
>>>> Regarding your attached screenshoot, i saw the files names from that view
>>>> and some of them they start with capital letters but in xml code they are
>>>> with small letters. Do you think that is the problem ?.  If so can you do
>>>> a
>>>> small test for me ?. Can you place the contents of the attached rar file
>>>> to
>>>> skins folder (rar file has the appropriated folder structure) and tell me
>>>> if
>>>> at least the view from your screenshoot It displays correctly ?.
>>>>
>>>> Thanks and sorry for all this mess.
>>>>
>>>> Nessus
>>>>
>>>> --------------------------------------------------
>>>> From: "Cory Fields" <[email protected]>
>>>> Sent: Saturday, April 21, 2012 9:39 AM
>>>>
>>>> To: "Nessus" <[email protected]>
>>>> Cc: <[email protected]>
>>>> Subject: Re: [Xbmc-addons] [Git Pull] Skin Xperience More
>>>>
>>>>> Ok, there are still some issues. I'm assuming that they're related to
>>>>> case-sensitivity issues, but I'm not seeing anything in the log. See
>>>>> attached as an example. Compare it to
>>>>>
>>>>> https://github.com/Nessus85100/Xperience.More/blob/317026f0cb5f9904a8394cadb898910a26fe29bc/_screenshots/6.jpg?raw=true
>>>>>
>>>>> Do you have a case-sensitive system (linux, osx w/ case-sensitive
>>>>> option, etc) you can use for testing?
>>>>>
>>>>> Regards,
>>>>> Cory
>>>>>
>>>>> On Wed, Apr 18, 2012 at 11:06 PM, Cory Fields <[email protected]> wrote:
>>>>>>
>>>>>> Those are were likely fixed when the animation xml's were cleaned up.
>>>>>> I'll give it another run-through and let you know for sure.
>>>>>>
>>>>>> Regards,
>>>>>> Cory
>>>>>>
>>>>>> On Wed, Apr 18, 2012 at 6:43 PM, Nessus <[email protected]>
>>>>>> wrote:
>>>>>>>
>>>>>>> Done. All issues are resolved.
>>>>>>> The only things that I cant find and fix them are these:
>>>>>>>
>>>>>>> 00:14:24 T:140536173619136    INFO: Loading skin file: MyVideoNav.xml
>>>>>>> 00:14:24 T:140536173619136 WARNING: Trying to add unsupported control
>>>>>>> type 3
>>>>>>> 00:14:24 T:140536173619136 WARNING: Previous line repeats 2 times.
>>>>>>> 00:14:24 T:140536173619136 WARNING: Trying to add unsupported control
>>>>>>> type 3
>>>>>>> 00:14:24 T:140536173619136   ERROR: Control has invalid animation type
>>>>>>> (no
>>>>>>> condition or no type)
>>>>>>> 00:14:24 T:140536173619136   ERROR: Previous line repeats 8 times.
>>>>>>> 00:14:24 T:140536173619136 WARNING: Trying to add unsupported control
>>>>>>> type 3
>>>>>>> 00:14:24 T:140536173619136 WARNING: Previous line repeats 3 times.
>>>>>>>
>>>>>>> If you can point me how and where to find them I will do them too.
>>>>>>>
>>>>>>> Version now is : v1.2.1
>>>>>>>
>>>>>>> Cheers
>>>>>>> Nessus
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> -----Original Message-----
>>>>>>> From: Cory Fields
>>>>>>> Sent: Wednesday, April 18, 2012 7:23 AM
>>>>>>> To: [email protected]
>>>>>>> Cc: Nessus ; [email protected]
>>>>>>> Subject: Re: [Xbmc-addons] [Git Pull] Skin Xperience More
>>>>>>>
>>>>>>> Ok, here we go. It's normal to have a bunch of issues to resolve when
>>>>>>> a skin is first submitted, so don't be alarmed by the list below. I'm
>>>>>>> certainly not.
>>>>>>>
>>>>>>> - revision aabb120 does not exist. So I'm reviewing current master
>>>>>>> (0edb3d)
>>>>>>>
>>>>>>> - please rename to skin.xperience-more or skin.xperience_more. The .
>>>>>>> and caps could cause us grief in the future.
>>>>>>>
>>>>>>> - version needs major.minor.revision. 1.2.0 would be fine.
>>>>>>>
>>>>>>> - please fix the xml's that don't validate. Most are just due to
>>>>>>> missing encoding indicators:
>>>>>>>
>>>>>>> $ find -name "*.xml" -exec xmlstarlet val "{}" \; | grep -i invalid
>>>>>>> ./Viewtype_Music.xml - invalid
>>>>>>> ./Viewtype_Posters music.xml - invalid
>>>>>>> ./script-globalsearch-main.xml - invalid
>>>>>>> ./PlayerControls.xml - invalid
>>>>>>> ./Includes_Animations.xml - invalid
>>>>>>> ./Viewtype_Posters_Big.xml - invalid
>>>>>>> ./Viewtype_Home_Panels.xml - invalid
>>>>>>> ./Font.xml - invalid
>>>>>>> ./Includes_Window-Contents.xml - invalid
>>>>>>> ./Includes_Home.xml - invalid
>>>>>>> ./Viewtype_Posters_Programs.xml - invalid
>>>>>>> ./script-XBMC-Subtitles-main.xml - invalid
>>>>>>> ./Viewtype_TV Show Banners.xml - invalid
>>>>>>> ./Viewtype_File Panel.xml - invalid
>>>>>>> ./DialogSelect.xml - invalid
>>>>>>> ./Viewtype_Walls_Full_Programs.xml - invalid
>>>>>>> ./Viewtype_Fanart_Tri_Panel.xml - invalid
>>>>>>> ./Viewtype_Slide.xml - invalid
>>>>>>> ./Viewtype_Slide_Programs.xml - invalid
>>>>>>> ./Custom_Settings.xml - invalid
>>>>>>> ./Viewtype_Walls_Full.xml - invalid
>>>>>>>
>>>>>>> - please no spaces in XML names. There are already a few cases of
>>>>>>> this, but it proves to be a headache during packaging and I'd rather
>>>>>>> not continue the trend.
>>>>>>>
>>>>>>> - missing screenshots. Please add a folder at the skin's root dir
>>>>>>> called "_screenshots" (with the underscore) and include around 10
>>>>>>> jpegs.
>>>>>>>
>>>>>>> - script.watchlist: If you're going to use it, you need to depend on
>>>>>>> it
>>>>>>>
>>>>>>> - Includes.xml -> includes.xml. This really makes me worry about other
>>>>>>> case-sensitive issues, the skin is entirely black in Linux due to the
>>>>>>> missing includes.
>>>>>>>
>>>>>>> - various issues during use, some of these may be resolved by making
>>>>>>> the xml's validate:
>>>>>>>
>>>>>>> 00:14:24 T:140536173619136    INFO: Loading skin file: MyVideoNav.xml
>>>>>>> 00:14:24 T:140536173619136 WARNING: Trying to add unsupported control
>>>>>>> type 3
>>>>>>> 00:14:24 T:140536173619136 WARNING: Previous line repeats 2 times.
>>>>>>> 00:14:24 T:140536173619136   ERROR: unmatched parentheses in
>>>>>>> SubString(ListItem.videoresolution,480
>>>>>>> 00:14:24 T:140536173619136   ERROR: unmatched parentheses in
>>>>>>> substring(listitem.VideoResolution,1080
>>>>>>> 00:14:24 T:140536173619136   ERROR: unmatched parentheses in
>>>>>>> substring(ListItem.filename,540p
>>>>>>> 00:14:24 T:140536173619136 WARNING: Trying to add unsupported control
>>>>>>> type 3
>>>>>>> 00:14:24 T:140536173619136   ERROR: Control has invalid animation type
>>>>>>> (no condition or no type)
>>>>>>> 00:14:24 T:140536173619136   ERROR: Previous line repeats 8 times.
>>>>>>> 00:14:24 T:140536173619136 WARNING: Trying to add unsupported control
>>>>>>> type 3
>>>>>>> 00:14:24 T:140536173619136 WARNING: Previous line repeats 3 times.
>>>>>>>
>>>>>>> Please let me know when those are worked out and I'll beat you up some
>>>>>>> more
>>>>>>> ;)
>>>>>>>
>>>>>>> Nice job on the skin, keep up the good work!
>>>>>>>
>>>>>>> Regards,
>>>>>>> Cory
>>>>>>>
>>>>>>> On Tue, Apr 17, 2012 at 2:43 AM, Cory Fields <[email protected]> wrote:
>>>>>>>>
>>>>>>>> Ok, That's good enough for me. I'll give it an initial review.
>>>>>>>>
>>>>>>>> Regards,
>>>>>>>> Cory
>>>>>>>>
>>>>>>>> On Sun, Apr 15, 2012 at 2:34 PM, Jezz_X <[email protected]>
>>>>>>>> wrote:
>>>>>>>>>
>>>>>>>>> Yeah I was also under the impression that team blackbolt were not
>>>>>>>>> working on this anymore or anything really
>>>>>>>>>
>>>>>>>>> On Sun, Apr 15, 2012 at 10:35 PM, Nessus <[email protected]>
>>>>>>>>> wrote:
>>>>>>>>>>
>>>>>>>>>> Original Xperience was abandoned more than a year now. Many times I
>>>>>>>>>> try
>>>>>>>>>> to
>>>>>>>>>> contact the project leader (XBMCG33K) but I didn’t get any answer.
>>>>>>>>>> If
>>>>>>>>>> you
>>>>>>>>>> manage to contact with him ask him if is ok otherwise no problem.
>>>>>>>>>>
>>>>>>>>>> Since is my favorite skin in the beginning i was trying to make the
>>>>>>>>>> skin
>>>>>>>>>> compatible with newer versions of XBMC but now I've rewrite the
>>>>>>>>>> code
>>>>>>>>>> and
>>>>>>>>>> create the graphics for more than the 70% of the skin.
>>>>>>>>>>
>>>>>>>>>> Xperience More v1.2
>>>>>>>>>> http://forum.xbmc.org/showthread.php?tid=103818
>>>>>>>>>>
>>>>>>>>>> Cheers
>>>>>>>>>> Nessus
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> --------------------------------------------------
>>>>>>>>>> From: "Cory Fields" <[email protected]>
>>>>>>>>>> Sent: Saturday, April 14, 2012 10:21 AM
>>>>>>>>>> To: "Nessus" <[email protected]>
>>>>>>>>>> Cc: <[email protected]>; "Team Blackbolt"
>>>>>>>>>> <[email protected]>; <[email protected]>
>>>>>>>>>> Subject: Re: [Xbmc-addons] [Git Pull] Skin Xperience More
>>>>>>>>>>
>>>>>>>>>>> On Fri, Apr 13, 2012 at 4:09 PM, Nessus <[email protected]>
>>>>>>>>>>> wrote:
>>>>>>>>>>>>
>>>>>>>>>>>> *addon -   skin.xperience.more
>>>>>>>>>>>> *version - 1.2
>>>>>>>>>>>> *url - git://github.com/Nessus85100/Xperience.More.git
>>>>>>>>>>>> *revision – aabb120
>>>>>>>>>>>> *branch - master
>>>>>>>>>>>> *xbmc version – eden
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>> ------------------------------------------------------------------------------
>>>>>>>>>>>> For Developers, A Lot Can Happen In A Second.
>>>>>>>>>>>> Boundary is the first to Know...and Tell You.
>>>>>>>>>>>> Monitor Your Applications in Ultra-Fine Resolution. Try it FREE!
>>>>>>>>>>>> http://p.sf.net/sfu/Boundary-d2dvs2
>>>>>>>>>>>> _______________________________________________
>>>>>>>>>>>> Xbmc-addons mailing list
>>>>>>>>>>>> [email protected]
>>>>>>>>>>>> https://lists.sourceforge.net/lists/listinfo/xbmc-addons
>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> Hmm, I'm a bit uncomfortable with this seeing as there's not one
>>>>>>>>>>> from
>>>>>>>>>>> the Blackbolt guys in yet.
>>>>>>>>>>>
>>>>>>>>>>> CCing for their opinion.
>>>>>>>>>>>
>>>>>>>>>>> Regards,
>>>>>>>>>>> Cory
>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> ------------------------------------------------------------------------------
>>>>>>>>>> For Developers, A Lot Can Happen In A Second.
>>>>>>>>>> Boundary is the first to Know...and Tell You.
>>>>>>>>>> Monitor Your Applications in Ultra-Fine Resolution. Try it FREE!
>>>>>>>>>> http://p.sf.net/sfu/Boundary-d2dvs2
>>>>>>>>>> _______________________________________________
>>>>>>>>>> Xbmc-addons mailing list
>>>>>>>>>> [email protected]
>>>>>>>>>> https://lists.sourceforge.net/lists/listinfo/xbmc-addons
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> ------------------------------------------------------------------------------
>>>>>>>>> For Developers, A Lot Can Happen In A Second.
>>>>>>>>> Boundary is the first to Know...and Tell You.
>>>>>>>>> Monitor Your Applications in Ultra-Fine Resolution. Try it FREE!
>>>>>>>>> http://p.sf.net/sfu/Boundary-d2dvs2
>>>>>>>>> _______________________________________________
>>>>>>>>> Xbmc-addons mailing list
>>>>>>>>> [email protected]
>>>>>>>>> https://lists.sourceforge.net/lists/listinfo/xbmc-addons
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> ------------------------------------------------------------------------------
>>>>>>> Better than sec? Nothing is better than sec when it comes to
>>>>>>> monitoring Big Data applications. Try Boundary one-second
>>>>>>> resolution app monitoring today. Free.
>>>>>>> http://p.sf.net/sfu/Boundary-dev2dev
>>>>>>> _______________________________________________
>>>>>>> Xbmc-addons mailing list
>>>>>>> [email protected]
>>>>>>> https://lists.sourceforge.net/lists/listinfo/xbmc-addons
>>>>
>>>>
>>>> ------------------------------------------------------------------------------
>>>> For Developers, A Lot Can Happen In A Second.
>>>> Boundary is the first to Know...and Tell You.
>>>> Monitor Your Applications in Ultra-Fine Resolution. Try it FREE!
>>>> http://p.sf.net/sfu/Boundary-d2dvs2
>>>> _______________________________________________
>>>> Xbmc-addons mailing list
>>>> [email protected]
>>>> https://lists.sourceforge.net/lists/listinfo/xbmc-addons
>>>>
>>>
>>
>> ------------------------------------------------------------------------------
>> For Developers, A Lot Can Happen In A Second.
>> Boundary is the first to Know...and Tell You.
>> Monitor Your Applications in Ultra-Fine Resolution. Try it FREE!
>> http://p.sf.net/sfu/Boundary-d2dvs2
>> _______________________________________________
>> Xbmc-addons mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/xbmc-addons
>
> ------------------------------------------------------------------------------
> For Developers, A Lot Can Happen In A Second.
> Boundary is the first to Know...and Tell You.
> Monitor Your Applications in Ultra-Fine Resolution. Try it FREE!
> http://p.sf.net/sfu/Boundary-d2dvs2
> _______________________________________________
> Xbmc-addons mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/xbmc-addons

------------------------------------------------------------------------------
For Developers, A Lot Can Happen In A Second.
Boundary is the first to Know...and Tell You.
Monitor Your Applications in Ultra-Fine Resolution. Try it FREE!
http://p.sf.net/sfu/Boundary-d2dvs2
_______________________________________________
Xbmc-addons mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/xbmc-addons

Reply via email to