You don't need to have email as a constant in your select for your cursor, just use the value that is already there.
The bigger question I would ask is do you really need a separate email for each sku or would a single email with all the skus be acceptable? If a single email with multiple skus would be ok you don't even need the cursor. And do you mean for these emails to be only a subject and not have a body?
--EDIT--
Meant to post the cleaned up version of your cursor.
DECLARE @emailTo VARCHAR(255),
@email VARCHAR(max),
@sku VARCHAR(30),
@emailSubject VARCHAR(128),
SET @emailTo = 'myemail@mycompany.co.uk'
SET @emailSubject = 'Warning stock is below safety for : '
-- SELECT RECORDS THAT NEED TO BE EMAILED
DECLARE c1 CURSOR FOR
SELECT rtrim(StockCode) as StockCode
FROMdbo.tbl_STOCK_safety_check
WHEREemail_status = 1
-- LOOP THROUGH RECORDSET AND SEND EMAIL FOR EACH RECORD
OPEN c1
FETCH NEXT FROM c1 INTO @sku
WHILE @@FETCH_STATUS <> -1
BEGIN
EXEC msdb.dbo.sp_send_dbmail
@importance = 'High',
@recipients= @email ,
@subject = @emailSubject + @sku
FETCH NEXT FROM c1 INTO @email,@sku
END
CLOSE c1
DEALLOCATE c1
_______________________________________________________________
Need help? Help us help you.
Read the article at http://www.sqlservercentral.com/articles/Best+Practices/61537/ for best practices on asking questions.
Need to split a string? Try Jeff Modens splitter http://www.sqlservercentral.com/articles/Tally+Table/72993/.
Cross Tabs and Pivots, Part 1 – Converting Rows to Columns - http://www.sqlservercentral.com/articles/T-SQL/63681/
Cross Tabs and Pivots, Part 2 - Dynamic Cross Tabs - http://www.sqlservercentral.com/articles/Crosstab/65048/
Understanding and Using APPLY (Part 1) - http://www.sqlservercentral.com/articles/APPLY/69953/
Understanding and Using APPLY (Part 2) - http://www.sqlservercentral.com/articles/APPLY/69954/