SageTV Community  

Go Back   SageTV Community > SageTV Development and Customizations > SageTV Github Development

Notices

SageTV Github Development Discussion related to SageTV Open Source Development. Use this forum for development topics about the Open Source versions of SageTV, hosted on Github.

Reply
 
Thread Tools Search this Thread Display Modes
  #1  
Old 12-30-2018, 08:09 PM
JREkiwi's Avatar
JREkiwi JREkiwi is offline
Sage Icon
 
Join Date: Jan 2005
Location: Auckland, New Zealand
Posts: 2,059
What brand of Line Endings?

Is there a consensus on what Line Endings should be used?

Visual Studio wants to use CR LF (Windows), but has the option of setting them to
CR (Macintosh)
LF (Unix)
LS (Unicode Line Separator)
PS (Unicode Paragraph Separator)

I would assume LF (Unix) is the way to go, but you know what happens when you assume

John
Reply With Quote
  #2  
Old 12-30-2018, 09:01 PM
fidget's Avatar
fidget fidget is offline
Sage Icon
 
Join Date: Jun 2003
Location: Cedar Rapids, Iowa, USA
Posts: 1,173
Most config management systems, like SVN, will do the line termination conversion for you, so Id recommend using whatever is native for your OS.
__________________
Server: i5-2405S (4 core @ 2.5 GHz), 8GB RAM, NORCO RPC-4220 4U case
Tuners: 2 SiliconDust HDHomeRun , 2 Hauppauge HD-PVR Connected to 1 Pace700X and 1 TiVo Series 4
DVD Storage: 24 TB
TV Storage: 11 TB (4x1.5TB for recording, 5TB for archive)
Clients: 3
SageTV Extenders:5
Reply With Quote
  #3  
Old 12-30-2018, 09:36 PM
wnjj wnjj is offline
Sage Icon
 
Join Date: Jan 2009
Posts: 1,052
If you're working with the SageTV code, some of the VS files have a mix within the same file and git complains about them but still manages them fine. The license header text was mass-copied in to all files and I assume has different line endings than the main code in some cases.

In general I would say VS files/projects should be CR/LF and the others just LF. That's mostly the case now.

Also, if you're planning on any code changes to SageTV, I'm just about ready for a pretty large pull request so please hold off or start from my github copy.
Reply With Quote
Reply


Currently Active Users Viewing This Thread: 1 (0 members and 1 guests)
 
Thread Tools Search this Thread
Search this Thread:

Advanced Search
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump

Similar Threads
Thread Thread Starter Forum Replies Last Post
Brand New HD300s heffneil SageTV Marketplace 0 08-17-2016 09:28 AM
WTS: BRAND NEW hd300's x 2 + 3 hd200s PiX64 SageTV Marketplace 8 08-02-2013 07:12 PM
Brand New HD 300 - No Sound WarlordBB SageTV HD Theater - Media Player 1 03-12-2011 07:42 PM
Not sure which brand of tuner to buy. jal Hardware Support 4 11-22-2009 08:15 AM
Brand new PVR-350 for sale mogrinz SageTV Software 0 02-13-2004 10:37 AM


All times are GMT -6. The time now is 11:16 AM.


Powered by vBulletin® Version 3.8.7
Copyright ©2000 - 2019, vBulletin Solutions, Inc.
Copyright 2003-2005 SageTV, LLC. All rights reserved.